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

Add transaction multiple patch method #50

Merged
merged 1 commit into from
Oct 19, 2021

Conversation

nuno-vieira-deel
Copy link
Contributor

This PR aims to add transaction multiple patch method.

Copy link
Contributor

@johnmaia johnmaia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nunovieira220 please take a look at my comments.

@@ -74,6 +74,16 @@ export default class TransactionManager extends AbstractManager {
return { results, total };
}

/**
* Multiple patch.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

* Patch multiple

* Multiple patch.
*/

async multiplePatch(data) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

patchMultiple

@@ -34,6 +34,10 @@ const config = {
endpoint: 'transactions',
method: 'get'
},
multiplePatch: {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

patchMultiple

@nuno-vieira-deel nuno-vieira-deel force-pushed the feature/transaction-multiple-patch branch from ee3b874 to 06c4b8b Compare October 15, 2021 16:46
@nuno-vieira-deel nuno-vieira-deel force-pushed the feature/transaction-multiple-patch branch from 06c4b8b to f5d07c9 Compare October 15, 2021 17:36
@joaonice joaonice merged commit f314ba3 into master Oct 19, 2021
@joaonice joaonice deleted the feature/transaction-multiple-patch branch October 19, 2021 14:01
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

Successfully merging this pull request may close these issues.

3 participants