shopinvader / odoo-shopinvader-payment

GNU Affero General Public License v3.0
2 stars 10 forks source link

[14.0][ADD] invader_payment_adyen_sepa #84

Closed acsonefho closed 1 year ago

acsonefho commented 1 year ago

Depends on https://github.com/shopinvader/odoo-shopinvader-payment/pull/80 (because this PR contains a lot of refactor needed for this feature).

When a debit order (SEPA) is created (and confirmed), a cron will get them and follow the process to create related transaction (in draft). Another cron will reach Adyen (only for sepadirectdebit) and try to capture the amount.

acsonefho commented 1 year ago

@Cedric-Pigeon last one :tada:

codecov-commenter commented 1 year ago

Codecov Report

Patch coverage: 67.32% and project coverage change: -0.60 :warning:

Comparison is base (5750e71) 76.78% compared to head (7f3e9a4) 76.18%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## 14.0 #84 +/- ## ========================================== - Coverage 76.78% 76.18% -0.60% ========================================== Files 106 115 +9 Lines 1555 1793 +238 Branches 185 217 +32 ========================================== + Hits 1194 1366 +172 - Misses 306 369 +63 - Partials 55 58 +3 ``` | [Impacted Files](https://app.codecov.io/gh/shopinvader/odoo-shopinvader-payment/pull/84?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=shopinvader) | Coverage Δ | | |---|---|---| | [invader\_payment\_adyen\_klarna/\_\_init\_\_.py](https://app.codecov.io/gh/shopinvader/odoo-shopinvader-payment/pull/84?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=shopinvader#diff-aW52YWRlcl9wYXltZW50X2FkeWVuX2tsYXJuYS9fX2luaXRfXy5weQ==) | `100.00% <ø> (ø)` | | | [...der\_payment\_adyen\_klarna/models/invader\_payable.py](https://app.codecov.io/gh/shopinvader/odoo-shopinvader-payment/pull/84?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=shopinvader#diff-aW52YWRlcl9wYXltZW50X2FkeWVuX2tsYXJuYS9tb2RlbHMvaW52YWRlcl9wYXlhYmxlLnB5) | `37.93% <0.00%> (ø)` | | | [...ent\_adyen\_klarna\_capture/models/invader\_payable.py](https://app.codecov.io/gh/shopinvader/odoo-shopinvader-payment/pull/84?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=shopinvader#diff-aW52YWRlcl9wYXltZW50X2FkeWVuX2tsYXJuYV9jYXB0dXJlL21vZGVscy9pbnZhZGVyX3BheWFibGUucHk=) | `23.17% <0.00%> (ø)` | | | [...der\_payment\_adyen\_klarna\_sale/models/sale\_order.py](https://app.codecov.io/gh/shopinvader/odoo-shopinvader-payment/pull/84?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=shopinvader#diff-aW52YWRlcl9wYXltZW50X2FkeWVuX2tsYXJuYV9zYWxlL21vZGVscy9zYWxlX29yZGVyLnB5) | `53.33% <ø> (ø)` | | | [...r\_payment\_adyen\_sepa/models/payment\_transaction.py](https://app.codecov.io/gh/shopinvader/odoo-shopinvader-payment/pull/84?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=shopinvader#diff-aW52YWRlcl9wYXltZW50X2FkeWVuX3NlcGEvbW9kZWxzL3BheW1lbnRfdHJhbnNhY3Rpb24ucHk=) | `40.38% <40.38%> (ø)` | | | [invader\_payment/models/invader\_payable.py](https://app.codecov.io/gh/shopinvader/odoo-shopinvader-payment/pull/84?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=shopinvader#diff-aW52YWRlcl9wYXltZW50L21vZGVscy9pbnZhZGVyX3BheWFibGUucHk=) | `48.27% <50.00%> (+0.65%)` | :arrow_up: | | [...ayment\_adyen\_klarna\_invoice/models/account\_move.py](https://app.codecov.io/gh/shopinvader/odoo-shopinvader-payment/pull/84?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=shopinvader#diff-aW52YWRlcl9wYXltZW50X2FkeWVuX2tsYXJuYV9pbnZvaWNlL21vZGVscy9hY2NvdW50X21vdmUucHk=) | `57.14% <50.00%> (+0.89%)` | :arrow_up: | | [...nvader\_payment\_adyen/models/payment\_transaction.py](https://app.codecov.io/gh/shopinvader/odoo-shopinvader-payment/pull/84?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=shopinvader#diff-aW52YWRlcl9wYXltZW50X2FkeWVuL21vZGVscy9wYXltZW50X3RyYW5zYWN0aW9uLnB5) | `59.37% <57.14%> (-3.13%)` | :arrow_down: | | [...payment\_adyen\_sepa/models/account\_payment\_order.py](https://app.codecov.io/gh/shopinvader/odoo-shopinvader-payment/pull/84?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=shopinvader#diff-aW52YWRlcl9wYXltZW50X2FkeWVuX3NlcGEvbW9kZWxzL2FjY291bnRfcGF5bWVudF9vcmRlci5weQ==) | `59.18% <59.18%> (ø)` | | | [...\_payment\_adyen\_sepa/models/account\_payment\_line.py](https://app.codecov.io/gh/shopinvader/odoo-shopinvader-payment/pull/84?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=shopinvader#diff-aW52YWRlcl9wYXltZW50X2FkeWVuX3NlcGEvbW9kZWxzL2FjY291bnRfcGF5bWVudF9saW5lLnB5) | `64.28% <64.28%> (ø)` | | | ... and [10 more](https://app.codecov.io/gh/shopinvader/odoo-shopinvader-payment/pull/84?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=shopinvader) | |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Cedric-Pigeon commented 1 year ago

/ocabot merge patch

shopinvader-git-bot commented 1 year ago

What a great day to merge this nice PR. Let's do it! Prepared branch 14.0-ocabot-merge-pr-84-by-Cedric-Pigeon-bump-patch, awaiting test results.

Cedric-Pigeon commented 1 year ago

/ocabot merge patch

shopinvader-git-bot commented 1 year ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 14.0-ocabot-merge-pr-84-by-Cedric-Pigeon-bump-patch, awaiting test results.

shopinvader-git-bot commented 1 year ago

Congratulations, your PR was merged at 44ec5d7218d4861cb88aac2a200ebace3e3f54df. Thanks a lot for contributing to shopinvader. ❤️

shopinvader-git-bot commented 1 year ago

@Cedric-Pigeon The merge process could not be finalized because an exception was raised: _find_issue() missing 1 required positional argument: 'target_branch'.