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

Port 7.0 add last sale price yvr #14

Merged
merged 8 commits into from
Sep 26, 2014

Conversation

yvaucher
Copy link
Member

No description provided.

@pedrobaeza
Copy link
Member

There is an error with this module tests. Can you check it, please?

Regards.

@yvaucher
Copy link
Member Author

last_sale_price tests fail because of sale_stock:

IntegrityError: null value in column "picking_policy" violates not-null constraint

This module doesn't depends on sale_stock, thus it should be tested separately

@gurneyalex
Copy link
Member

@yvaucher I think you can work around this by changing the travis configuration for the repository, in order to make separate tests for the modules which must not be tested with sale_stock installed.

@yvaucher
Copy link
Member Author

Separating last_sale_price test works though it seems there is still an error with sale_validity on other test. (this shouldn't have anything to do with last_sale_price)

@eLBati
Copy link
Member

eLBati commented Aug 25, 2014

The test error came from #22
@yvaucher you can try rebasing

@coveralls
Copy link

Coverage Status

Coverage increased (+26.17%) when pulling 74f632d on yvaucher:port-7.0-add-last_sale_price-yvr into 6ef5ea8 on OCA:7.0.

@yvaucher yvaucher force-pushed the port-7.0-add-last_sale_price-yvr branch from 74f632d to 9a784f1 Compare August 25, 2014 20:15
@yvaucher
Copy link
Member Author

Rebased

@eLBati
Copy link
Member

eLBati commented Aug 26, 2014

The remaining failure comes from OCA/OCB#37
So, 👍

@gurneyalex gurneyalex force-pushed the port-7.0-add-last_sale_price-yvr branch from 9a784f1 to 0e26095 Compare September 25, 2014 11:42
@gurneyalex gurneyalex force-pushed the port-7.0-add-last_sale_price-yvr branch from 0e26095 to 1d7614a Compare September 25, 2014 13:16
@gurneyalex
Copy link
Member

rebased on top of 7.0

@coveralls
Copy link

Coverage Status

Coverage increased (+1.09%) when pulling 1d7614a on yvaucher:port-7.0-add-last_sale_price-yvr into e66bbfe on OCA:7.0.

@gurneyalex
Copy link
Member

👍

1 similar comment
@pedrobaeza
Copy link
Member

👍

pedrobaeza added a commit that referenced this pull request Sep 26, 2014
@pedrobaeza pedrobaeza merged commit ec6b57f into OCA:7.0 Sep 26, 2014
@yvaucher yvaucher deleted the port-7.0-add-last_sale_price-yvr branch October 3, 2014 09:10
rousseldenis pushed a commit to acsone/sale-workflow that referenced this pull request Jul 30, 2019
…w_payment_mode_setup-dro

[12.0][IMP] sale_automatic_workflow_payment_mode: setup
lmignon pushed a commit to acsone/sale-workflow that referenced this pull request Feb 11, 2021
cluster picking: implement /stock_issue
lmignon pushed a commit to acsone/sale-workflow that referenced this pull request Feb 11, 2021
Signed-off-by guewen
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.

None yet

5 participants