Skip to content

Conversation

@mihien
Copy link

@mihien mihien commented Dec 16, 2025

Change the structure of the address block in the xml to be compliant with the newest SEPA guidelines (as of dec 2025).

This is heavily based on #1174 , keeping just the bare minimum of this solution to have the new structured address as instructed in the EPC guidance 2.1.

@mihien mihien force-pushed the 16.0-imp-sepa-structured-address branch 2 times, most recently from bb1d55d to 0e0c3b3 Compare December 16, 2025 09:23
@pedrobaeza pedrobaeza added this to the 16.0 milestone Dec 16, 2025
class ResPartner(models.Model):
_inherit = "res.partner"

def _generate_address_block(self, parent_node, gen_args):
Copy link
Member

Choose a reason for hiding this comment

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

As you are moving this method, please rename it to something understandable on its own inside res.partner, like _generate_sepa_pain_address_block.

ctry.text = self.country_id.code

@api.model
def _improved_street_split(self, street):
Copy link
Member

Choose a reason for hiding this comment

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

The same for this.

Copy link
Author

Choose a reason for hiding this comment

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

Good point, I've changed the two names (and one more) to make them more explicit.

@mihien mihien force-pushed the 16.0-imp-sepa-structured-address branch 4 times, most recently from f2135bc to 4e2bc5b Compare December 16, 2025 16:42
@mihien mihien marked this pull request as draft December 16, 2025 16:50
mihien and others added 2 commits December 24, 2025 16:58
Remove support for:
- SCT : pain.001.001.04, pain.001.001.05
- SDD : ain.008.001.03, ain.008.001.04
because these PAIN version have never been adopted by EPC.
Update and improve README
@mihien mihien force-pushed the 16.0-imp-sepa-structured-address branch from 4e2bc5b to 284a9d8 Compare December 24, 2025 16:00
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