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

[17.0][MIG] account_analytic_distribution_manual: Migration to 17.0 #700

Draft
wants to merge 40 commits into
base: 17.0
Choose a base branch
from

Conversation

BernatObrador
Copy link
Contributor

@BernatObrador BernatObrador commented Oct 10, 2024

Module migrated to version 17.0

cc https://github.com/APSL 161694

@miquelalzanillas @lbarry-apsl @mpascuall @peluko00 @javierobcn @ppyczko please review

…complete analytic accounts on any model that has a field for analytic accounts.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_distribution_manual
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_distribution_manual/
Currently translated at 100.0% (23 of 23 strings)

Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_distribution_manual
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_distribution_manual/it/
Currently translated at 95.6% (22 of 23 strings)

Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_distribution_manual
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_distribution_manual/es/
…tics account from manual distribution

- Show the current distribution selected at the top of the tags
- When unselecting a distribution, all related analytic accounts are also unselected
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_distribution_manual
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_distribution_manual/
Currently translated at 100.0% (26 of 26 strings)

Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_distribution_manual
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_distribution_manual/it/
…teger to many2one

- If the user needs to group by this field, it was impossible when it was an integer.
- Added a script to populate the new field in models related to account_analytic_tags.
Note: When multiple tags are present, only the last record is populated.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_distribution_manual
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_distribution_manual/
Currently translated at 100.0% (26 of 26 strings)

Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_distribution_manual
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_distribution_manual/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_distribution_manual
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_distribution_manual/
Currently translated at 100.0% (27 of 27 strings)

Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_distribution_manual
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_distribution_manual/it/
…export/import of analytical distributions

TT50235
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_distribution_manual
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_distribution_manual/
Currently translated at 100.0% (29 of 29 strings)

Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_distribution_manual
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_distribution_manual/it/
victoralmau and others added 5 commits October 9, 2024 15:14
…export/import to analytic.mixin

Move the functionality to the analytic.mixin model so that it can also be used in account.move.line records

TT50944
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-analytic-16.0/account-analytic-16.0-account_analytic_distribution_manual
Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-account_analytic_distribution_manual/
@BernatObrador BernatObrador force-pushed the 17.0-mig-account_analytic_distribution_manual branch from 5febd1f to 5ddf248 Compare October 11, 2024 11:40
@BernatObrador BernatObrador mentioned this pull request Oct 15, 2024
18 tasks
@BernatObrador BernatObrador force-pushed the 17.0-mig-account_analytic_distribution_manual branch 6 times, most recently from 527842e to 4a11424 Compare October 18, 2024 06:11
@BernatObrador
Copy link
Contributor Author

Hi @pedrobaeza!

I hope you're doing well! I’m facing some issues with the tests as they’re failing due to the tour, and I'm not sure how to resolve it. If you could lend a hand, I would really appreciate it!

Thank you in advance!

@pedrobaeza
Copy link
Member

Not sure. We had a similar problem some weeks ago that was solved with this: OCA/oca-ci#78

@sbidoul @chienandalu do you have any clue?

@BernatObrador
Copy link
Contributor Author

Thanks, @pedrobaeza! If there's anything I can assist with, just let me know.

Copy link
Member

@chienandalu chienandalu left a comment

Choose a reason for hiding this comment

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

You need to change the tour first

@BernatObrador BernatObrador force-pushed the 17.0-mig-account_analytic_distribution_manual branch 2 times, most recently from be2a906 to 1c2324c Compare November 7, 2024 09:06
@BernatObrador
Copy link
Contributor Author

Hi @chienandalu! Thank you very much for your contribution! The changes have been applied, but now the tour fails at the step "Create new invoice" with the trigger .o_list_button_add.

I'll take a closer look at this and see if I can resolve it. Thanks again for your help!

@chienandalu
Copy link
Member

Yes, now it's just a matter of debugging the steps 😉

@BernatObrador BernatObrador force-pushed the 17.0-mig-account_analytic_distribution_manual branch from 1c2324c to f87beaa Compare November 7, 2024 10:29
@BernatObrador BernatObrador marked this pull request as draft November 7, 2024 10:29
@BernatObrador BernatObrador force-pushed the 17.0-mig-account_analytic_distribution_manual branch from f87beaa to 379c043 Compare November 7, 2024 10:37
@BernatObrador BernatObrador force-pushed the 17.0-mig-account_analytic_distribution_manual branch from 379c043 to 158bf67 Compare November 22, 2024 09:26
@BernatObrador BernatObrador force-pushed the 17.0-mig-account_analytic_distribution_manual branch from 158bf67 to e90815d Compare November 22, 2024 09:29
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.

9 participants