-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[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
base: 17.0
Are you sure you want to change the base?
Conversation
task-4737738 task-4794170 task-4360579
f85fe0e
to
dd8d039
Compare
There was a problem hiding this 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 🙈 )
content/applications/finance/accounting/customer_invoices/electronic_invoicing.rst
Show resolved
Hide resolved
content/applications/finance/accounting/customer_invoices/electronic_invoicing.rst
Show resolved
Hide resolved
content/applications/finance/accounting/customer_invoices/electronic_invoicing.rst
Show resolved
Hide resolved
content/applications/finance/accounting/customer_invoices/electronic_invoicing.rst
Show resolved
Hide resolved
content/applications/finance/accounting/customer_invoices/electronic_invoicing.rst
Show resolved
Hide resolved
content/applications/finance/accounting/customer_invoices/electronic_invoicing.rst
Show resolved
Hide resolved
content/applications/finance/accounting/customer_invoices/electronic_invoicing.rst
Show resolved
Hide resolved
content/applications/finance/accounting/customer_invoices/electronic_invoicing.rst
Show resolved
Hide resolved
content/applications/finance/accounting/customer_invoices/electronic_invoicing.rst
Show resolved
Hide resolved
|
||
.. 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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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)
task-4737738
task-4794170
task-4360579