Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Documentation in README to clarify how this project relates to Fineract's API client #29

Open
vorburger opened this issue Aug 25, 2020 · 2 comments

Comments

@vorburger
Copy link
Member

@Grandolf49 perhaps it would be useful to add something to the top of the https://github.com/openMF/fineract-client/blob/master/README.md which clarifies how this project relates to https://github.com/apache/fineract/blob/develop/docs/developers/swagger/client.md ?

Because it's actually not clear e.g. to me (and perhaps to others, in the future). From what little I understand, this looks like it may be a Git repo containing the output of the code that one can generate by following https://github.com/apache/fineract/blob/develop/docs/developers/swagger/client.md, is that right? But you seem to also be making manual extensions here (like #11), and letting Dependabot upgrade many 3rd-parties (which, ideally, should better be made "upstream" in Swagger Codegen, which would then release new versions, which Apache Fineract would, eventually, pick up through Dependabot upgrades of Swagger Codegen).

@Grandolf49
Copy link
Collaborator

Grandolf49 commented Aug 26, 2020

@vorburger Sure, I'll add more details about this repository.

For you information:

  1. This repository is more focused on the Android Applications that Mifos uses. Hence, some manual extensions were required here. Whereas the library generated in apache-fineract will be more generic and not Android Specific. Perhaps, I should consider modifying the name of this repository.

  2. Yes, this is a project generated by Swagger Codegen. I have used Swagger 2.0 API Spec File, which differs from the latest spec file OpenAPI 3.0.0 generated in Apache Fineract project.

  3. Reason for using old Spec File is because android-client and mifos-mobile and mobile-wallet android applications are using Fineract 1.0.0 (which I understand is a very old version of Fineract).

  4. If anyone of the Android Project mentioned above require some fixes to be done in this library, we can perform that directly here. It is quite possible that these fixes need human intervention which Mustache templates might not solve.

I would love to have feedback of @vorburger @droidchef @therajanmaurya @shiv07tiwari @edcable for deciding how to proceed here.

vorburger added a commit to apache/fineract that referenced this issue Jan 7, 2022
@vorburger
Copy link
Member Author

I've added now a link to the "other" (official) fineract-client to the README of this project via 8061350.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants