-
-
Notifications
You must be signed in to change notification settings - Fork 715
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
[MIG][9.0] product_sequence #156
Conversation
needs label change to "needs review" |
👍 |
Usage | ||
===== | ||
|
||
#. Go to ... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please include short usage instructions
Please include some tests. |
c92c088
to
9cb8dbb
Compare
# -*- coding: utf-8 -*- | ||
# © 2004 Tiny SPRL | ||
# © 2016 Sodexis | ||
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). | ||
|
||
from . import models | ||
from .models.product_product import update_null_and_slash_codes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should change this also to a post/pre_init_hook.
Please update .travis.yml definition to get a correct Travis status |
9cb8dbb
to
9d20322
Compare
@pedrobaeza corrections done |
Why the coverage drop? You can test the init_hook creating a product, emptying by SQL the code, executing the hook and checking that is filled with the expected value. |
@pedrobaeza added test and fixed your points. COVERAGE DECREASED (-0.9%) TO 94.382%.. This seems alright to me :) |
No description provided.