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

[18.0] pending prs #1

Open
wants to merge 27 commits into
base: master
Choose a base branch
from
Open

[18.0] pending prs #1

wants to merge 27 commits into from

Conversation

LucasTran380381
Copy link
Collaborator

@LucasTran380381 LucasTran380381 commented Nov 25, 2024

inlude PRs:

  • [ADD] odoo_module_migrate: Deprecated kanban-box-card-menu https://github.com/OCA/odoo-module-migrator/pull/119
  • [IMP] replace toggle_button to boolean_toggle widget https://github.com/OCA/odoo-module-migrator/pull/118
  • [IMP] remove string attribute in tree view https://github.com/OCA/odoo-module-migrator/pull/117
  • [IMP] *: add warning for auto-added invisible/readonly fields in XML views https://github.com/OCA/odoo-module-migrator/pull/116
  • [IMP] *: deprecate ustr() https://github.com/OCA/odoo-module-migrator/pull/115
  • [IMP] _() improvements https://github.com/OCA/odoo-module-migrator/pull/114
  • [IMP] Add warning for direct '_' import in translations https://github.com/OCA/odoo-module-migrator/pull/113
  • [IMP] Add error messages for deprecated '_check_recursion' https://github.com/OCA/odoo-module-migrator/pull/112
  • [IMP] Add error messages for deprecated '_name_search' method, replac… https://github.com/OCA/odoo-module-migrator/pull/111
  • [IMP] Add error messages for deprecated 'registry' usage and imports https://github.com/OCA/odoo-module-migrator/pull/110
  • [IMP] automatically remove @odoo-module from js assets https://github.com/OCA/odoo-module-migrator/pull/108
  • [IMP] replace editable attribute on list view https://github.com/OCA/odoo-module-migrator/pull/107
  • [ADD] renamed fields case for l10n_mx https://github.com/OCA/odoo-module-migrator/pull/106
  • [IMP] slugify deprecation https://github.com/OCA/odoo-module-migrator/pull/105
  • [ADD] Add error message chart_template_ref argument has been removed in v18 https://github.com/OCA/odoo-module-migrator/pull/101
  • [IMP] simplify kanban archs https://github.com/OCA/odoo-module-migrator/pull/121
  • [ADD] replace action.read() by _for_xml_id to avoid access rights issue https://github.com/OCA/odoo-module-migrator/pull/76

anothingguy and others added 23 commits October 26, 2023 17:32
…d `kanban_color`, `kanban_getcolor`, `kanban_getcolorname`
…refore the get_files method returns some files wrongly
…k_m2m_recursion' methods, replaced by '_has_cycle'

More info here: odoo/odoo#162656
Replace deprecated on view kanb-box, kanban-card, kanban-menu
@nilshamerlinck
Copy link

can you please also include:

  • https://github.com/OCA/odoo-module-migrator/pull/121 ;-)
  • https://github.com/OCA/odoo-module-migrator/pull/76

and update https://github.com/OCA/odoo-module-migrator/pull/105 that has changed a bit meanwhile (you might have noticed the error already)

@LucasTran380381
Copy link
Collaborator Author

LucasTran380381 commented Nov 26, 2024

can you please also include:

  • https://github.com/OCA/odoo-module-migrator/pull/121 ;-)
  • https://github.com/OCA/odoo-module-migrator/pull/76

and update https://github.com/OCA/odoo-module-migrator/pull/105 that has changed a bit meanwhile (you might have noticed the error already)

I'm done, could you check it again.

more question:

  • I will check pull request from OCA repo periodically, and include new PRs ?
  • In your opinion, rebase or merge is good to include PRs? ( I think rebase more clear commit history but have painful git conflict -> could you share your tips)

@nilshamerlinck
Copy link

thanks!

  • yes, that's a good idea if you can keep an eye on the new PRs and keep this branch updated; notify it on #trobz-oca-migration-process` whenever you update it also so that we would use latest changes
  • given that this branch is just for internal usage, merge is good enough for me

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