Skip to content

[IMP] account_peppol: skip recompute peppol if already registered. - #5283

Open
YousefAllam221b wants to merge 1 commit into
19.0-onboarding-september-malbfrom
19.0-skip-recompute-peppol-yalla
Open

[IMP] account_peppol: skip recompute peppol if already registered.#5283
YousefAllam221b wants to merge 1 commit into
19.0-onboarding-september-malbfrom
19.0-skip-recompute-peppol-yalla

Conversation

@YousefAllam221b

Copy link
Copy Markdown

When someone is registred on Peppol, we fill his Peppol EAS/Endpoint. But if he later fills his VAT, we recompute these fields, resulting in a broken config. Now, already registered partners are skipped and only unregisered are recomputed

task-6517921

When someone is registred on Peppol, we fill his Peppol EAS/Endpoint. But if he later fills his VAT, we recompute these fields, resulting in a broken config. Now, already registered partners are skipped and only unregisered are recomputed

task-6517921
@robodoo

robodoo commented Sep 4, 2026

Copy link
Copy Markdown

This PR targets the un-managed branch odoo-dev/odoo:19.0-onboarding-september-malb, it needs to be retargeted before it can be merged.

@malb-odoo malb-odoo 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.

Good job ! This is exactly what was merge actually (apart from the test).

I see that you use the exact same bank account that was used in the solution :suspect:. I would prefer that you ask questions instead of looking at the solution when you are blocked 😅. When you will join a subteam you will not have any solution to help you.

@YousefAllam221b

Copy link
Copy Markdown
Author

Good job ! This is exactly what was merge actually (apart from the test).

I see that you use the exact same bank account that was used in the solution :suspect:. I would prefer that you ask questions instead of looking at the solution when you are blocked 😅. When you will join a subteam you will not have any solution to help you.

I havent looked at the solution, the bank account was the same from the previous function in the same file, and the rest of the infomration was filled by the autocomplete (but i didnt have any complains with it so i left them as is).

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