-
-
Notifications
You must be signed in to change notification settings - Fork 426
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
[15.0][IMP] account_statement_import_online_paypal: new paypal event_codes #644
[15.0][IMP] account_statement_import_online_paypal: new paypal event_codes #644
Conversation
Hi @alexey-pelykh, |
@ntsirintanis Thanks for the PR! Is there a chance you'd be willing to port that across the versions? Also, is there a |
Hoi Alexey, yes, there actually is:
Though I am not quite sure if this is relevant to Odoo. Nevertheless, I can add another key. |
@ntsirintanis if that's not too troublesome, please add - there are a lot of keys that are probably irrelevant yet they are there. I'll fast-track the changes anyways |
|
b27ebaf
to
9c493e7
Compare
Hi @alexey-pelykh , I've pushed more codes in all PRs |
@ntsirintanis thanks! I guess there's no reason having 2 commits for this, just squash them - otherwise looks awesome |
9c493e7
to
d439e43
Compare
This PR has the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/ocabot merge patch
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at ada56b9. Thanks a lot for contributing to OCA. ❤️ |
Related: #642