[FEATURE] - Add endpoint to update transaction data#596
[FEATURE] - Add endpoint to update transaction data#596CalvinChanCan wants to merge 6 commits intomintapi:mainfrom
Conversation
|
I'm still thinking on this one. We've so far maintained the stance that this is simply a read-only library. That being said, I do get that if this is a sought after feature and we have contributors willing to do the work, then that stance may need to be reconsidered. If we do, I think we should work on major version v3 that supports a variety of update actions instead of slow trickling them in with various v2 minor versions. |
|
Thanks, this looks cool! I don't have any strong opinions. As Matthew said, previously we've maintained a read-only stance, partially because I would be concerned about there being enough folks available and interested to maintain it. That said, it is probably easier with the REST endpoints. I think my preferred approach would be a new repo under the same project umbrella named |
|
Thanks @mrooney . I'm unsure if I have enough experience to create a new module. Specifically is the goal of the new module to try to create a wrapper around mintapi with write endpoints? |
|
Apologies for the long delay @CalvinChanCan ! If you are still willing to work on this and a separate module seems overwhelming, I would be fine with just separating files, so that we have an |
Not sure if this is allowed and feel free to close if not.
This PR adds an endpoint to update an existing transaction with an updated description, notes, category, and tags and to retrieve the updated transaction.
I appreciate any feedback and guidance.