Skip to content

[18.0][MIG] account_banking_pain_base sepa hybrid extended for 09#1553

Open
ntsirintanis wants to merge 1 commit intoOCA:18.0from
Therp:18.0-mig-account_banking_pain_base_sepa_hybrid_extended_for_09
Open

[18.0][MIG] account_banking_pain_base sepa hybrid extended for 09#1553
ntsirintanis wants to merge 1 commit intoOCA:18.0from
Therp:18.0-mig-account_banking_pain_base_sepa_hybrid_extended_for_09

Conversation

@ntsirintanis
Copy link
Copy Markdown
Contributor

related: #1550

@ntsirintanis
Copy link
Copy Markdown
Contributor Author

@thomaspaulb

@thomaspaulb
Copy link
Copy Markdown

@pedrobaeza Is the cherry-pick OK or do we actually need to jump through the hoop of the "OCA technical migration method" now that the 14.0 PR has become merged?

@pedrobaeza pedrobaeza added this to the 18.0 milestone Feb 9, 2026
Copy link
Copy Markdown
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

I think both commits can be squashed, and more when there are changes in the migration script that doesn't belong to migration itself (like strip thing, that sound like a refinement found during the migration).

@thomaspaulb
Copy link
Copy Markdown

Ok, but do we need to rebase on 14.0 and follow the technical migration method?

@pedrobaeza
Copy link
Copy Markdown
Member

No, because this is not a full module, but a new feature. In fact, this shouldn't be qualified as migration, but a forward-port of a [IMP], being itself [IMP]. Remove [14.0] from the commit message.

@ntsirintanis ntsirintanis force-pushed the 18.0-mig-account_banking_pain_base_sepa_hybrid_extended_for_09 branch from 580d1b5 to 518541d Compare February 9, 2026 13:35
@thomaspaulb
Copy link
Copy Markdown

Ah of course yes, don't know where my brain was with that question.

@pedrobaeza
Copy link
Copy Markdown
Member

Please squash both commits into one.

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.

3 participants