8000 [FW][IMP] localization: Add Chorus pro via Peppol information by fw-bot · Pull Request #13199 · odoo/documentation · GitHub
[go: up one dir, main page]
Skip to content

[FW][IMP] localization: Add Chorus pro via Peppol information #13199

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

[FW][IMP] localization: Add Chorus pro via Peppol information #13199

wants to merge 1 commit into from

Conversation

fw-bot
Copy link
Collaborator
@fw-bot fw-bot commented May 5, 2025

task-4698357
task-4661523

For your info,

  • Wynand confirmed that the "VAT Anti-Fraud Certification" module is only for POS.
  • I'll have to make some more changes for 18.0 (including task 4661523)
    Thank you!

Forward-Port-Of: #13130
Forward-Port-Of: #12897

@robodoo
Copy link
Collaborator
robodoo commented May 5, 2025

Pull request status dashboard

@fw-bot
Copy link
Collaborator Author
fw-bot commented May 5, 2025

@afma-odoo @xpl-odoo cherrypicking of pull request #12897 failed.

stdout:

Auto-merging content/applications/finance/accounting/customer_invoices/electronic_invoicing.rst
Auto-merging content/applications/finance/fiscal_localizations/france.rst
CONFLICT (content): Merge conflict in content/applications/finance/fiscal_localizations/france.rst

Either perform the forward-port manually (and push to this branch, proceeding as usual) or close this PR (maybe?).

In t 8000 he former case, you may want to edit this PR message as well.

⚠️ after resolving this conflict, you will need to merge it via @robodoo.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@afma-odoo afma-odoo force-pushed the 18.0-17.0-accounting-localization-france-chorus-pro-afma-432552-fw branch from 4c935f8 to 7c94dbb Compare May 7, 2025 10:36
@C3POdoo C3POdoo requested review from a team May 7, 2025 10:43
@afma-odoo afma-odoo requested review from a team and auva-odoo and removed request for a team and auva-odoo May 7, 2025 10:47
@afma-odoo afma-odoo force-pushed the 18.0-17.0-accounting-localization-france-chorus-pro-afma-432552-fw branch from 7c94dbb to b1534ab Compare May 8, 2025 08:16
@afma-odoo afma-odoo requested a review from auva-odoo May 8, 2025 08:31
auva-odoo
auva-odoo approved these changes May 9, 2025
View reviewed changes
Copy link
Contributor
@auva-odoo auva-odoo left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Great work on this fw-port PR, @afma-odoo! You’ve clearly put a lot of effort into reviewing everything thoroughly, thank you! I just have minor comments 😉

content/applications/finance/fiscal_localizations/france.rst Outdated

.. _localization/france/fec-import:
- :guilabel:`eInvoice format`: Select :guilabel:`BIS Billing 3.0`.
- Select :guilabel:`France SIRET`, then type `11000201100044`, the reference used by Chorus Pro.
Copy link
Contributor

Choose a reason for hiding this comment

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

France SIRET seems to be the default option so I would maybe rephrase this?

Suggested change
- Select :guilabel:`France SIRET`, then type `11000201100044`, the reference used by Chorus Pro.
- Make sure :guilabel:`France SIRET` is selected in the next field, then type `11000201100044`, the reference used by Chorus Pro.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, sure! You're right :)

content/applications/finance/fiscal_localizations/france.rst
year, you might need to cancel those entries in the User Interface. Odoo makes those entries
(RAN) useless.
#. Confirm the invoice.
#. Click :guilabel:`Send` and, in the :guilabel:`Print & Send` window, enable :guilabel:`By Peppol`.
Copy link
Contributor

Choose a reason for hiding this comment

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

I accidentally tested this on 18.1 and it seems the window's label is again Send from 18.1? 🙈 just FYI.

Copy link
Contributor

Choose a reason for hiding this comment

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

Thank you for the info 🤗

@afma-odoo afma-odoo force-pushed the 18.0-17.0-accounting-localization-france-chorus-pro-afma-432552-fw branch from b1534ab to 7779dab Compare May 12, 2025 07:16
@afma-odoo afma-odoo requested a review from auva-odoo May 12, 2025 07:45
@fw-bot
Copy link
Collaborator Author
fw-bot commented May 13, 2025

@afma-odoo @xpl-odoo this forward port of #12897 is awaiting action (not merged or closed).

auva-odoo
auva-odoo approved these changes May 13, 2025
View reviewed changes
Copy link
Contributor
@auva-odoo auva-odoo left a comment

Choose a reason for hiding this comment

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

LGTM @afma-odoo 🙂 : (you don't need me to r+ since this is a fw-port PR 😉 )

@afma-odoo
Copy link
Contributor

@auva-odoo Thank you for your review ☺️

@robodoo r+

@robodoo robodoo closed this in ab4b704 May 13, 2025
This was referenced May 13, 2025
Open
Open
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@afma-odoo afma-odoo afma-odoo left review comments

@auva-odoo auva-odoo auva-odoo approved these changes

Assignees
No one assigned
Labels
conflict forwardport
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@fw-bot @robodoo @afma-odoo @auva-odoo
0