Skip to content
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

[MIG] account_journal_general_sequence: v16 modifications #1536

Merged
merged 17 commits into from
Dec 23, 2022

Conversation

yajo
Copy link
Member

@yajo yajo commented Dec 23, 2022

Nothing special.

@moduon MT-1921

yajo and others added 17 commits December 23, 2022 10:22
Under some legislations, account moves must follow a single sequence. Since Odoo removed this sequence number in recent versions, this information was lost.

With this module, you can force your account moves to follow a separate sequence. The sequence is automatic when a move is posted. Includes a wizard to reorder those numbers in the sequence.

@moduon MT-676
…m zero

This is a special case when working with `ir.sequence`.

@moduon MT-676
On databases with big amounts of account moves, installation would freeze Odoo for some minutes.

We skip now entry number computation at install, to avoid such cases.

@moduon MT-676
…gers renumber moves

Before this patch, if users were accounting managers but not system administrators, they couldn't execute the account move renumbering.

Now that's fixed and tested.

@moduon MT-1728
@yajo
Copy link
Member Author

yajo commented Dec 23, 2022

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

Sorry @yajo you are not allowed to merge.

To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons.

If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the maintainers key of its manifest.

@yajo
Copy link
Member Author

yajo commented Dec 23, 2022

@OCA/accounting-maintainers could somebody please merge this? I'm the maintainer but I can't do it myself because of this bug in the bot: OCA/oca-github-bot#226

Copy link
Member

@rafaelbn rafaelbn left a comment

Choose a reason for hiding this comment

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

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-1536-by-rafaelbn-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 09e91c0 into OCA:16.0 Dec 23, 2022
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at c2594cc. Thanks a lot for contributing to OCA. ❤️

@yajo yajo deleted the 16.0-account_journal_general_sequence-mig branch December 23, 2022 12:13
@rafaelbn
Copy link
Member

/ocabot migration account_journal_general_sequence

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Dec 23, 2022
@OCA-git-bot OCA-git-bot mentioned this pull request Dec 23, 2022
33 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants