-
Notifications
You must be signed in to change notification settings - Fork 46
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
[API v.2] Selection of multiple categories #21
Comments
It's a simple code whis no multicategory support. https://github.com/moneymanagerex/moneymanagerex/blob/master/src/webapp.cpp#L404-L411 Form a JSON with an array for split. I think it is not difficult to implement this in the near future. |
Offtopic @vomikan What's the numbering approach that you use? I thought of "major.minor.patch" where "patch" is incremented for hot fixes. While the "minor" would go for regular updates. And the next version would be 1.1.0 but you've incremented the last digit, not the middle. /Offtopic |
@vomikan thank you for desktop code reference. Few more Q:
|
I suggest the amount field should be the total of the transaction. Somthing like this:
yes |
Hello @maksimdzmitryew I think that in this case (and transfer amount) you should issue a new API version on PHP side and @vomikan should do the same on desktop API side (you have to agree on the fields type etc..). Everything is already managed in desktop version, so until the webapp stays at the expected version of the desktop, everything works. Then user will have to update both at the same time (there should be a bit of coordination between developers to have both at the same time). Personally I think that issue a new API version is the best way if supporting the old code means having lots of tricks or not linear code. |
@gabriele-v thank you for your input. I appreciate you shared your thoughts. It will definitely help |
Hi @vomikan
At the moment, I can't select more than 1 category+subcategory when filling in new transaction data. I would like to implement this on the WebApp UI but need to make sure this is supported by Desktop API.
Please, advise.
The text was updated successfully, but these errors were encountered: