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.

Sign up for GitHub

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

Jump to bottom

[12.0][ADD] account_bank_statement_import_online_transferwise #220

Merged
merged 1 commit into from
Mar 26, 2020
Merged

[12.0][ADD] account_bank_statement_import_online_transferwise #220

merged 1 commit into from
Mar 26, 2020

Conversation

alexey-pelykh
Copy link
Contributor

@alexey-pelykh alexey-pelykh commented Aug 2, 2019 •
edited
Loading

Adds automatic pulling of statements from Transferwise

@pedrobaeza pedrobaeza added this to the 12.0 milestone Aug 2, 2019
@alexey-pelykh alexey-pelykh force-pushed the 12.0-add-account_bank_statement_import_online_transferwise branch 29 times, most recently from 36010bb to 90ecfd9 Compare September 13, 2019 05:28
@alexey-pelykh alexey-pelykh force-pushed the 12.0-add-account_bank_statement_import_online_transferwise branch from 7b1bbd3 to 0ac86aa Compare September 25, 2019 10:43
@alexey-pelykh alexey-pelykh force-pushed the 12.0-add-account_bank_statement_import_online_transferwise branch from 0ac86aa to 402dc02 Compare September 26, 2019 14:55
@alexey-pelykh alexey-pelykh force-pushed the 12.0-add-account_bank_statement_import_online_transferwise branch 5 times, most recently from 00ff6f9 to a4512cf Compare November 3, 2019 11:23
@alexey-pelykh alexey-pelykh force-pushed the 12.0-add-account_bank_statement_import_online_transferwise branch 5 times, most recently from f0db2e4 to f560b80 Compare November 14, 2019 14:15
@alexey-pelykh alexey-pelykh force-pushed the 12.0-add-account_bank_statement_import_online_transferwise branch from f560b80 to 8c902a9 Compare February 12, 2020 15:40
artemsenko
artemsenko approved these changes Feb 12, 2020
View reviewed changes
Copy link

@artemsenko artemsenko left a comment

Choose a reason for hiding this comment

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

functional tests

sweenya
sweenya approved these changes Feb 13, 2020
View reviewed changes
Copy link

@sweenya sweenya left a comment

Choose a reason for hiding this comment

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

Functional test

@OCA-git-bot
Copy link
Contributor

Sorry @alexey-pelykh you are not allowed to merge.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

@alexey-pelykh
Copy link
Contributor Author

@OCA/banking-maintainers would it be possible to make further steps about this module?

YegorOkhotnikov
YegorOkhotnikov approved these changes Mar 23, 2020
View reviewed changes
Copy link

@YegorOkhotnikov YegorOkhotnikov left a comment

Choose a reason for hiding this comment

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

Functional test

@alexey-pelykh
Copy link
Contributor Author

As we've tested this for quite long internally and on our installations, and there are no objections, I'll merge it

@alexey-pelykh
Copy link
Contributor Author

/ocabot merge

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 12.0-ocabot-merge-pr-220-by-alexey-pelykh-bump-no, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 99ec214 into OCA:12.0 Mar 26, 2020
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at c87e22d. Thanks a lot for contributing to OCA. ❤️

@alexey-pelykh alexey-pelykh deleted the 12.0-add-account_bank_statement_import_online_transferwise branch March 26, 2020 07:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@artemsenko artemsenko artemsenko approved these changes

@sweenya sweenya sweenya approved these changes

@YegorOkhotnikov YegorOkhotnikov YegorOkhotnikov approved these changes

@astirpe astirpe Awaiting requested review from astirpe

@sbidoul sbidoul Awaiting requested review from sbidoul

Assignees
No one assigned
Labels
merged 🎉
Projects
None yet
Milestone
12.0
Development

Successfully merging this pull request may close these issues.
7 participants
@alexey-pelykh @astirpe @OCA-git-bot @artemsenko @sweenya @YegorOkhotnikov @pedrobaeza