Skip to content

Conversation

@bosd
Copy link

@bosd bosd commented Nov 16, 2025

WIP, Should be functional. Commits are a bit of a mess. some fixes should have been backported in the v18.

But various fixes already applied here.

Based upon #225

@bosd bosd force-pushed the 19.0-mig-attribute_set branch from d3445da to 963440b Compare November 18, 2025 07:56
@bosd bosd force-pushed the 19.0-mig-attribute_set branch from 14d00bc to 6ab37c5 Compare November 26, 2025 22:11
@bosd bosd marked this pull request as ready for review November 26, 2025 22:11
@bosd
Copy link
Author

bosd commented Nov 26, 2025

/ocabot migration

@OCA-git-bot
Copy link
Contributor

Hi @bosd. Your command failed:

Invalid options for command migration: .

Ocabot commands

  • ocabot merge major|minor|patch|nobump
  • ocabot rebase
  • ocabot migration {MODULE_NAME}

More information

@bosd bosd mentioned this pull request Nov 27, 2025
Copy link
Member

@rvalyi rvalyi left a comment

Choose a reason for hiding this comment

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

please do a rebase to use the 18.0 branch as we just merged #192 . Make sure you rebase your 19.0 migration atop of this 18.0 "official" migration. Sorry for the delay.

@rvalyi rvalyi closed this Dec 20, 2025
@rvalyi rvalyi reopened this Dec 20, 2025
clementmbr and others added 17 commits December 21, 2025 12:07
New organization :

- attribute_set (former base_custom_attribute)
- product_attribute_set (former pim_custom_attribute but without menus)
- pim (The "PIM application" former pim_base)
- pim_attrubute_set (depends on product_attribute_set and adds menus in
the PIM application)
And small FIX in _build_attribute_field()
[UPD] README.rst

[UPD] README.rst
…d attribute_set [CHG] add required columns on attributes on attribute set form view

attribute_set 13.0.1.0.1
sbejaoui and others added 10 commits December 21, 2025 12:07
As model_id is referring to non accessible models for
common users, set a related field in order to be able
to use it in views domains.

Moreover, add a domain on attribute_set_id field in
attribute owner mixin in order to simplify inherited
views.
As attribute view building is accessing ir.model records,
do it with sudo.

[UPD] Update attribute_set.pot

[BOT] post-merge updates

[UPD] Update attribute_set.pot
Currently translated at 100.0% (125 of 125 strings)

Translation: odoo-pim-16.0/odoo-pim-16.0-attribute_set
Translate-URL: https://translation.odoo-community.org/projects/odoo-pim-16-0/odoo-pim-16-0-attribute_set/es/
@bosd bosd force-pushed the 19.0-mig-attribute_set branch 5 times, most recently from af862b1 to 4b109b0 Compare December 21, 2025 12:58
bosd added 3 commits December 21, 2025 14:02
- Fix attribute_set native field updates during module upgrades
- Fix Attribute record duplication to ensure unique names
- Fix Attribute error where bool has no startswith method
- Fix attribute set placeholder handling to improve compatibility
- Fix popup attributes view to show more than 2 characters
- Fix view styling and other compatibility improvements
@bosd bosd force-pushed the 19.0-mig-attribute_set branch from 4b109b0 to 474ea47 Compare December 21, 2025 13:03
@bosd
Copy link
Author

bosd commented Dec 21, 2025

Done, it is now based ontop of the offical 18.0 migration ✨
i hope we can get the new momentum going in this repo 😉

@bosd
Copy link
Author

bosd commented Dec 21, 2025

/ocabot migration attribute_set

@OCA-git-bot
Copy link
Contributor

Sorry @bosd you are not allowed to mark the addon to be migrated.

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.

Copy link

@Dranyel-Bosd Dranyel-Bosd left a comment

Choose a reason for hiding this comment

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

LGTM 💯

@MiquelRForgeFlow
Copy link

@bosd Could you redo commit history? New language commit was added in v18.

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.