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 partner_area_data and partner_area_management #46

Merged
merged 3 commits into from
Jun 2, 2015

Conversation

Ehtaga
Copy link

@Ehtaga Ehtaga commented Apr 21, 2015

This module was written to enter a location's area based on different units of measure
The information is dependant on the different locations entered

This module was written to enter a location's area based on different units of measure
The information is dependant on the different locations entered
@bwrsandman bwrsandman added this to the 8.0 milestone Apr 22, 2015
@bwrsandman
Copy link

👍

@pedrobaeza
Copy link
Member

Some things:

  • Why partner_area? You are mentioning stock locations, so at least stock prefix should be in the name.
  • First module doesn't contain models, but only translations.

@oca-clabot
Copy link

Hey @Ehtaga, thank you for your Pull Request.

It looks like some users haven't signed our Contributor License Agreement, yet.
You can read and sign our full Contributor License Agreement here: http://odoo-community.org/page/website.cla
Here is a list of the users:

  • @Ehtaga (login unknown in OCA database)

Appreciation of efforts,
OCA CLAbot

@Ehtaga
Copy link
Author

Ehtaga commented Apr 22, 2015

Ok for the name.

The first module adds surface units of measure (m² and feet²). They are necessary to set a default value in the second module. This way, the second module depends on the first one and the data is loaded before the model.

@bwrsandman
Copy link

Can we merge?

@pedrobaeza
Copy link
Member

It's OK for me.

pedrobaeza added a commit that referenced this pull request Jun 2, 2015
Add partner_area_data and partner_area_management
@pedrobaeza pedrobaeza merged commit 5060d2f into OCA:8.0 Jun 2, 2015
@oca-clabot
Copy link

Hey @Ehtaga,
We acknowledge that the following users have signed our Contributor License Agreement:

Appreciation of efforts,
OCA CLAbot

gschrott-osi pushed a commit to ursais/stock-logistics-warehouse that referenced this pull request Jun 5, 2023
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.

5 participants