Skip to content

[IMP] pos: update Invoices documentation#17310

Open
emmi-odoo wants to merge 1 commit into19.0from
19.0-pos-invoices-emmi
Open

[IMP] pos: update Invoices documentation#17310
emmi-odoo wants to merge 1 commit into19.0from
19.0-pos-invoices-emmi

Conversation

@emmi-odoo
Copy link
Copy Markdown
Contributor

task-5177183

@robodoo
Copy link
Copy Markdown
Collaborator

robodoo commented Apr 9, 2026

Pull request status dashboard

@C3POdoo C3POdoo requested a review from a team April 9, 2026 14:16
@emmi-odoo emmi-odoo requested review from a team and removed request for a team April 9, 2026 15:01
@auva-odoo auva-odoo requested review from auva-odoo and removed request for a team April 10, 2026 11:16
Copy link
Copy Markdown
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.

@emmi-odoo I think a lot of information is missing from the PR, namely:

  1. How we can reprint/resend an invoice (can this be done both from the fronted and the backend? How?)
  2. Can we do that even if the Invoice check box wasn't enabled on the payment screen
  3. Can we create and send the invoice (for the first time) after the payment has been processed? I think this can be done:
  • from the frontend (from the Orders view, filter by Paid orders, then click Invoice in the pad on the right to download the invoice, but I dont't think you can send it from there, to be checked!)
  • from the backend: we could probably reference the Accounting doc for this process, but it should still be mentionned somewhere
  1. Given this list, it might be worth it to check if other things are not missing as well

The doc will probably need to be restructured a bit accordingly as well, given the above and the following elements:

  • Would it make sense to have a heading for the section where you explain how to create a global invoice for a specific customer? TBD
  • I definitely think we need a Reprint/resend section, that's probably a common need?

Also, I've mentioned this a couple of times now, so I want to be more explicit: please test the features you're documenting, especially when updating older content. Submitting without testing creates a back-and-forth that slows everyone down and makes the whole process less efficient than it needs to be (and more frustrating for everyone). It would really help if this became a habit before submitting for review. 🙏

Comment thread content/applications/sales/point_of_sale/use/pos_invoices.rst Outdated
Comment thread content/applications/sales/point_of_sale/use/pos_invoices.rst
Comment thread content/applications/sales/point_of_sale/use/pos_invoices.rst Outdated
Comment thread content/applications/sales/point_of_sale/use/pos_invoices.rst Outdated
Comment thread content/applications/sales/point_of_sale/use/pos_invoices.rst Outdated
Comment thread content/applications/sales/point_of_sale/use/pos_invoices.rst Outdated
Comment thread content/applications/sales/point_of_sale/use/pos_invoices.rst Outdated
Comment thread content/applications/sales/point_of_sale/use/pos_invoices.rst Outdated
Comment thread content/applications/sales/point_of_sale/use/pos_invoices.rst Outdated
Comment thread content/applications/sales/point_of_sale/use/pos_invoices.rst Outdated
@emmi-odoo emmi-odoo force-pushed the 19.0-pos-invoices-emmi branch from 675cf61 to 8a8161d Compare April 15, 2026 10:11
@emmi-odoo emmi-odoo force-pushed the 19.0-pos-invoices-emmi branch from 8a8161d to 75b908c Compare April 15, 2026 10:37
@emmi-odoo
Copy link
Copy Markdown
Contributor Author

Hello @auva-odoo , thank your for your review. Here is the update.

Comment thread content/applications/sales/point_of_sale/use/pos_invoices.rst
Comment on lines +21 to +25
To define the default journals for a specific POS, go to :menuselection:`Point of Sale -->
Configuration --> Settings`, scroll down to the :guilabel:`Accounting` section, and select the
appropriate journals for :guilabel:`Orders` and :guilabel:`Invoices` under :guilabel:`Default
Journals`. To modify the selected default journal, click the :icon:`fa-arrow-right`
(:guilabel:`Internal link`) icon.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Did you see and understand my previous comment about this? I don't see any reaction or reply to that comment so I'm not sure

Comment on lines +70 to +72
To generate a single global invoice for all orders linked to the same customer that are not yet
invoiced (i.e., invoices with an :guilabel:`Invoice Status` set to :guilabel:`To Invoice`), follow
these steps:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think we should reframe this as generating invoices (whether one per SO or a consolidated one) for multiple sales orders; the consolidation part is just an option IMO:

Suggested change
To generate a single global invoice for all orders linked to the same customer that are not yet
invoiced (i.e., invoices with an :guilabel:`Invoice Status` set to :guilabel:`To Invoice`), follow
these steps:
To generate invoices for multiple orders from the same customer that are not yet
invoiced (i.e., invoices with an :guilabel:`Invoice Status` set to :guilabel:`To Invoice`), follow
these steps:
  • note 1: the concept of generating invoices for multiple sales orders is not specific to POS but since it's not documented anywhere in the Accounting doc at the moment, we'll leave it here (there's a task to update the Accounting doc and they'll update this page when they get to it)
  • note 2: the part about not yet invoiced could be removed if you decide to add it in the introduction under the main heading Invoice generation

Comment thread content/applications/sales/point_of_sale/use/pos_invoices.rst
Comment thread content/applications/sales/point_of_sale/use/pos_invoices.rst
Comment thread content/applications/sales/point_of_sale/use/pos_invoices.rst
Comment thread content/applications/sales/point_of_sale/use/pos_invoices.rst
To create and send an invoice to a customer on the :ref:`Payment <pos/use/sell>` screen, follow
these steps:

#. Enable :icon:`fa-file-text-o` :guilabel:`Invoice`.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is there a way to make this automatic? I don't think you answered this question in my first review.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

From what I see, you can send automatic invoice follow-ups (which occur after this first invoice), but there don't appear to be any features to automate invoice sending when selecting a customer.

Comment thread content/applications/sales/point_of_sale/use/pos_invoices.rst
Comment thread content/applications/sales/point_of_sale/use/pos_invoices.rst
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