Skip to content

[IMP] accounting: update peppol information on e-invoicing page #13922

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.

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

Open
wants to merge 1 commit into
base: 17.0
Choose a base branch
from

Conversation

afma-odoo
Copy link
Contributor

task-4737738
task-4794170
task-4360579

@robodoo
Copy link
Collaborator

robodoo commented Jul 3, 2025

Pull request status dashboard

@C3POdoo C3POdoo requested review from a team July 3, 2025 08:53
@afma-odoo afma-odoo removed request for a team July 3, 2025 08:54
@afma-odoo afma-odoo force-pushed the 17.0-accounting-peppol-afma branch from f85fe0e to dd8d039 Compare July 3, 2025 09:25
@afma-odoo afma-odoo requested a review from a team July 3, 2025 11:15
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.

Hello @afma-odoo thank you for tackling this task, it was much, much needed!
In addition to my review comments:

  • The Demo/Live/Test radio buttons were removed so I don't think "demo mode", "live mode", etc. need to use guilabel and be capitalized? I mentioned this a few times in my review comments but you might want to check the whole Peppol section just in case?
  • Unless it is a UI element (and using a guilabel), I don't think the word "endpoint" needs to have a capital e. Could you please chekc the whole doc?

Thank you again for your work! (and your patience while I was figuring out the testing 🙈 )


.. image:: electronic_invoicing/peppol-fetch-bills.png
:alt: Fetch bills from Peppol

.. note::
When multiple purchase journals are used, incoming Peppol documents appear in the Document app
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
When multiple purchase journals are used, incoming Peppol documents appear in the Document app
When multiple purchase journals are used, incoming Peppol documents appear in the Documents app

(and could we link to the that doc page?)
(cf our discussion, to check that this is indeed for 17.0 as well (since there is no module like there is for 18.0)

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