[19.0][ADD] product_class: classes to group product attributes#2273
[19.0][ADD] product_class: classes to group product attributes#2273Ricardoalso wants to merge 1 commit into
Conversation
Ricardoalso
force-pushed
the
product_class
branch
from
96216f0 to
8c641f8
Compare
Ricardoalso
force-pushed
the
product_class
branch
from
8c641f8 to
2b3aece
Compare
Ricardoalso
force-pushed
the
product_class
branch
from
2b3aece to
ec42e77
Compare
ivantodorovich
left a comment
ivantodorovich
left a comment
There was a problem hiding this comment.
LG.
Small comments, pre-approving
Ricardoalso
force-pushed
the
product_class
branch
from
7486878 to
ef484ff
Compare
|
This PR has the |
simahawk
left a comment
simahawk
left a comment
There was a problem hiding this comment.
minor remark and a question: have you evaluated using attribute sets from OCA/pim?
We haven't, but my understanding is that it for a different use case |
After taking a look at the module's description, it seems to be a far more generic module where we could potentially achieve the same things (if we set aside the view overrides done in this product_class), but with a lot more configuration tuning. 🤔 This module is designed to be "plug and play" after installation and has a deterministic scope. But I appreciate you sharing and pointing out this module 🙏 |
|
@ivantodorovich as discussed, some new improvements pushed 50ff6fa |
vvrossem
left a comment
vvrossem
left a comment
There was a problem hiding this comment.
Thank you for the detailed DESCRIPTION.md and USAGE.md 👏
This module introduces Product Classes to group product attributes and standardize product setup.
6 participants
This module introduces Product Classes to group product attributes and standardize product setup.