Skip to content

OD-2177/automate adyen bank statment import#6

Open
mileomai2008 wants to merge 2 commits intoOD-2150/adyen_new_payment_link_backportfrom
OD-2177/automate_adyen_bank_statment_import
Open

OD-2177/automate adyen bank statment import#6
mileomai2008 wants to merge 2 commits intoOD-2150/adyen_new_payment_link_backportfrom
OD-2177/automate_adyen_bank_statment_import

Conversation

@mileomai2008
Copy link

@mileomai2008 mileomai2008 commented Oct 6, 2022

Description of the issue/feature this PR addresses:

Current behavior before PR: Doesn't handle 'REPORT_AVAILABLE' webhook

Desired behavior after PR is merged:

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@mileomai2008 mileomai2008 force-pushed the OD-2177/automate_adyen_bank_statment_import branch from cd7a2d4 to 9f3ad91 Compare October 6, 2022 07:52
@mileomai2008 mileomai2008 marked this pull request as draft October 6, 2022 07:53
@mileomai2008 mileomai2008 reopened this Oct 6, 2022
@mileomai2008 mileomai2008 marked this pull request as ready for review October 6, 2022 07:54
@mileomai2008 mileomai2008 marked this pull request as draft October 6, 2022 07:54
@mileomai2008 mileomai2008 marked this pull request as ready for review October 11, 2022 11:52

Choose a reason for hiding this comment

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

this should cause conflict wthi this https://github.com/VanMoof/odoo/pull/5/files#diff-5c0cadcc807ef523fe18a7db4ef532cafab74bbf5a083f7c1a55e0b5ee3e4ff7R53

Maybe a good time to merge all cusomizatins for Adyen on odoo/odoo repository and bump up the hash?

Choose a reason for hiding this comment

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

I dont see where we consume the report that comes back from Adyen?
I also dont see tests in our addons to cover this functionality.

@dkubiak789 dkubiak789 force-pushed the OD-2177/automate_adyen_bank_statment_import branch from 80314fd to 6ed8ac9 Compare October 19, 2022 10:54
@dkubiak789 dkubiak789 force-pushed the OD-2177/automate_adyen_bank_statment_import branch from 2b099e4 to daabc2d Compare October 26, 2022 21:41
@dkubiak789 dkubiak789 requested a review from MPishpecki October 27, 2022 08:27
@dkubiak789 dkubiak789 force-pushed the OD-2177/automate_adyen_bank_statment_import branch from daabc2d to 187bb3e Compare November 4, 2022 07:37
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.

4 participants