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

[ADD] product_profile module #65

Merged
merged 15 commits into from
Nov 27, 2015
Merged

Conversation

bealdav
Copy link
Member

@bealdav bealdav commented Apr 13, 2015

This module was written to make products configuration easier (in one click).
It allows to configure a product with only one field.

Profile dependents fields are hidden by default, UI is simplest

@bealdav
Copy link
Member Author

bealdav commented Apr 24, 2015

Travis is OK, Runbot failed because of product_sequence module which can't apply its contrainst with demo data of other modules. Then all is ok for this module


.. image:: static/description/field.png

**Main use case**: a lot of modules are installed (mrp, pruchase, sale, pos)
Copy link
Member

Choose a reason for hiding this comment

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

s/pruchase/purchase/

@gurneyalex
Copy link
Member

@bealdav typos to be fixed and 1 bad bug to fix before this can be merged.

@gurneyalex
Copy link
Member

(very nice and useful module, I like the idea and the implementation).

@bealdav
Copy link
Member Author

bealdav commented Apr 24, 2015

@gurneyalex thanks a lot for your complete review.
I have just introduced example module to have more feedback on the reliability of the first module.

@gurneyalex
Copy link
Member

👍 once the build is 🍏

@coveralls
Copy link

Coverage Status

Coverage increased (+19.33%) to 64.66% when pulling 36eae35 on akretion:8-product-profile into 082aac9 on OCA:8.0.

@bealdav
Copy link
Member Author

bealdav commented Apr 28, 2015

Impossible to see runbot log result. 404 error on main links :-(

@bealdav bealdav closed this May 9, 2015
@bealdav bealdav reopened this May 9, 2015
@bealdav bealdav closed this May 18, 2015
@bealdav bealdav reopened this May 18, 2015
@bealdav bealdav closed this May 23, 2015
'name': 'Product Profile',
'version': '8.0.1.0.0',
'author': 'Akretion, Odoo Community Association (OCA)',
'summarize': "Set a configuration profile to product templates",

Choose a reason for hiding this comment

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

summary

@bealdav
Copy link
Member Author

bealdav commented Nov 20, 2015

@elicoidal if you are ok, +1 please, thanks

@mourad-ehm
Copy link
Contributor

👍

#. module: product_profile
#: field:product.profile,sequence:0
msgid "Sequence"
msgstr "Sequence"

Choose a reason for hiding this comment

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

Séquence

@elicoidal
Copy link

👍 (no code review though)

@bealdav bealdav force-pushed the 8-product-profile branch 4 times, most recently from 144310c to c493eb2 Compare November 23, 2015 08:08
@bealdav
Copy link
Member Author

bealdav commented Nov 23, 2015

@jgrandguillaume , it seems there are sufficient +1, and like you're PSC of the branch, thanks to look at. Thanks

gurneyalex added a commit that referenced this pull request Nov 27, 2015
@gurneyalex gurneyalex merged commit 1e68a33 into OCA:8.0 Nov 27, 2015
@bealdav bealdav deleted the 8-product-profile branch January 11, 2016 19:43
hurrinico pushed a commit to hurrinico/product-attribute that referenced this pull request May 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.