Skip to content

[IMP] accounting: clarify mass-sequencing highlights and errors #12926

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-4725955

Update of both Invoice sequence & Vendor bill sequence

@robodoo
Copy link
Collaborator

robodoo commented Apr 17, 2025

Pull request status dashboard

@C3POdoo C3POdoo requested a review from a team April 17, 2025 14:25
Copy link
Contributor

@dade-odoo dade-odoo left a comment

Choose a reason for hiding this comment

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

Hey @afma-odoo thanks for the quick turnaround on this. I have one question for you regarding the time it takes for the red text of the first invoice/bill of a new sequence to disappear.
I can't comment on it since you didn't change it, but I also think we should remove "following" in line 6 of each doc, so the sentence remains

By default, it uses the sequence format BILL/year/month/incrementing-number (e.g.,
BILL/2025/01/00001), which restarts from 00001 each year.

.. note::
To indicate where the sequence change began, the first invoice in the new sequence is
highlighted in red in the :guilabel:`Customer Invoices` list. This visual marker is temporary
and will disappear automatically over time.
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you know how much time? Have you asked a PO about this?

@afma-odoo afma-odoo force-pushed the 17.0-accounting-mass-resequencing-highlights-afma branch from 9b316be to 05060e3 Compare April 18, 2025 09:52
@afma-odoo
Copy link
Contributor Author

@dade-odoo Thanks for your review and for catching the error about the non-temporary highlight ☺️
I added info about the message in the journal on the dashboard in case of irregularities in the new sequence.

@afma-odoo afma-odoo requested a review from dade-odoo April 18, 2025 10:14
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