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

[10.0][MIG] product_by_supplier #361

Merged
merged 2 commits into from
Apr 4, 2017

Conversation

LoisRForgeFlow
Copy link
Contributor

The changes included in this migration are:

  • Basic migration actions (odoo tags, bump version, templates...).
  • Update the README.
  • Fix moved external IDs.

@LoisRForgeFlow
Copy link
Contributor Author

cc @eficent

@pedrobaeza pedrobaeza mentioned this pull request Mar 14, 2017
38 tasks
Copy link
Contributor

@gmeficent gmeficent left a comment

Choose a reason for hiding this comment

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

Functional review 👍

Copy link
Contributor

@AaronHForgeFlow AaronHForgeFlow left a comment

Choose a reason for hiding this comment

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

It works as expected 👍

Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Little redaction bugs.

This module add a new menu item 'Product by suppliers' in Purchases -> Products.
It allows for users to be able to view a compiled list of products supplied by the supplier.
This information is the corresponding to the supplier list on each product.
This module add a new menu item 'Product by suppliers' in Purchases ->
Copy link
Member

Choose a reason for hiding this comment

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

s/add/adds

It allows for users to be able to view a compiled list of products supplied by the supplier.
This information is the corresponding to the supplier list on each product.
This module add a new menu item 'Product by suppliers' in Purchases ->
Products. It allows for users to be able to view a compiled list of products
Copy link
Member

Choose a reason for hiding this comment

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

Remove for

This information is the corresponding to the supplier list on each product.
This module add a new menu item 'Product by suppliers' in Purchases ->
Products. It allows for users to be able to view a compiled list of products
supplied by the supplier. This information is the corresponding to the
Copy link
Member

Choose a reason for hiding this comment

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

s/the supplier/their supplier

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've attended these mistakes.

@pedrobaeza pedrobaeza merged commit 50fbc7a into OCA:10.0 Apr 4, 2017
@MiquelRForgeFlow MiquelRForgeFlow deleted the 10.0-mig-product_by_supplier branch June 14, 2017 14:03
rgarnau pushed a commit to ForgeFlow/purchase-workflow that referenced this pull request Aug 31, 2017
tschanzt pushed a commit to camptocamp/purchase-workflow that referenced this pull request May 4, 2018
cubells pushed a commit to Tecnativa/purchase-workflow that referenced this pull request Sep 28, 2018
[11.0] product_brand: complete fr translation
LoisRForgeFlow added a commit to ForgeFlow/purchase-workflow that referenced this pull request Oct 18, 2018
LoisRForgeFlow added a commit to ForgeFlow/purchase-workflow that referenced this pull request Oct 18, 2018
LoisRForgeFlow added a commit to ForgeFlow/purchase-workflow that referenced this pull request Oct 18, 2018
LoisRForgeFlow added a commit to ForgeFlow/purchase-workflow that referenced this pull request Oct 18, 2018
pedrobaeza pushed a commit to ForgeFlow/purchase-workflow that referenced this pull request Oct 23, 2018
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.

4 participants