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] Modules moved from wip #5

Merged
merged 2 commits into from
Dec 13, 2014
Merged

[ADD] Modules moved from wip #5

merged 2 commits into from
Dec 13, 2014

Conversation

oihane
Copy link
Contributor

@oihane oihane commented Dec 11, 2014

  • mrp_bom_notes
  • mrp_production_notes
  • product_packaging_views
  • stock_lote_notes

@coveralls
Copy link

Coverage Status

Coverage increased (+17.14%) when pulling 9449011 on oihane:8.0-notes into c53d004 on odoomrp:8.0.

@pedrobaeza
Copy link
Member

Mira que nos han pasado a la nueva API mrp_production_notes. Pon esa nueva versión

@oihane
Copy link
Contributor Author

oihane commented Dec 11, 2014

y los renombro? solo con note en el nombre técnico??? tambien habría que pasar el de stock_lot_notes a la nueva api no?

😉

@pedrobaeza
Copy link
Member

Sí, se podría pasar sobre la marcha. Lo de renombrarlo, de momento déjalo a ver qué decidimos.

@coveralls
Copy link

Coverage Status

Coverage increased (+17.14%) when pulling e3976b9 on oihane:8.0-notes into c53d004 on odoomrp:8.0.

@oihane
Copy link
Contributor Author

oihane commented Dec 11, 2014

Ready

@pedrobaeza
Copy link
Member

Estoy viendo que los campos se llaman note, siendo incoherente con lo que hemos estado diciendo.

@oihane
Copy link
Contributor Author

oihane commented Dec 11, 2014

Bueno... hay de todo 😄

@pedrobaeza
Copy link
Member

Deberíamos unificarlos...

@oihane
Copy link
Contributor Author

oihane commented Dec 11, 2014

notes or note???

@pedrobaeza
Copy link
Member

notes

@oihane
Copy link
Contributor Author

oihane commented Dec 12, 2014

Cambios realizados

@coveralls
Copy link

Coverage Status

Coverage increased (+17.14%) when pulling 4b916be on oihane:8.0-notes into c53d004 on odoomrp:8.0.

@oihane
Copy link
Contributor Author

oihane commented Dec 12, 2014

Cuando se mergee este también hay que mergear odoomrp/odoomrp-wip#444

@pedrobaeza
Copy link
Member

Se te olvidó renombrar las traducciones, pero ya lo hago yo en otro commit.

pedrobaeza added a commit that referenced this pull request Dec 13, 2014
[ADD] Modules moved from wip
@pedrobaeza pedrobaeza merged commit ab86a58 into odoomrp:8.0 Dec 13, 2014
@oihane oihane deleted the 8.0-notes branch December 15, 2014 08:12
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

3 participants