Skip to content

[18.0][FIX] subscription_oca: Adapt test after Odoo Peppol update - #1396

Open
tobiaszehntner wants to merge 1 commit into
OCA:18.0from
acsone:18.0-fix-subscription-oca
Open

tobiaszehntner wants to merge 1 commit into
OCA:18.0from
acsone:18.0-fix-subscription-oca

Conversation

@tobiaszehntner

Copy link
Copy Markdown
Contributor

After Odoo Peppol security issue and latest updates:

Fix failing test: when posting invoices, the bank needs to be trusted.

@luisDIXMIT luisDIXMIT left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@rrebollo rrebollo left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review. LGTM! Would you be so kind to review my #1410 in return?

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions Bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Aug 9, 2026
@tobiaszehntner
tobiaszehntner force-pushed the 18.0-fix-subscription-oca branch from 6aa5310 to 1fc452a Compare August 10, 2026 12:01
@tobiaszehntner

Copy link
Copy Markdown
Contributor Author

@luisDIXMIT rebased to latest. could this be merged?

@github-actions github-actions Bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Aug 16, 2026

@qgroulard qgroulard left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM (code review)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants