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

Ecuador's geopolitical organization. #14

Merged
merged 8 commits into from
Nov 19, 2016
Merged

Ecuador's geopolitical organization. #14

merged 8 commits into from
Nov 19, 2016

Conversation

damendieta
Copy link

Adds the information of all provinces (states), canton and parish of Ecuador.

@coveralls
Copy link

coveralls commented Nov 10, 2016

Coverage Status

Coverage remained the same at 81.818% when pulling 303219b on damendieta:ote into 826f93d on OCA:9.0.

@coveralls
Copy link

coveralls commented Nov 10, 2016

Coverage Status

Coverage remained the same at 81.818% when pulling 316d56e on damendieta:ote into 826f93d on OCA:9.0.

Ecuador's Geopolitical Information
==================================

This module adds the information requiered to select partner's location in states, canton and parish.
Copy link
Sponsor Member

Choose a reason for hiding this comment

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

required

To install this module, you need to:

* Add the module to your addons path.
* Install the module as usual.
Copy link
Sponsor Member

Choose a reason for hiding this comment

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

No need for this section if it's the normal process.

Configuration
=============

This module doesn't requiere any configuration.
Copy link
Sponsor Member

Choose a reason for hiding this comment

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

No need for this section if there is nothing to configure.


This module changes the structure in partner views to give you the options to select the cantons and parish, just use them.

Demostración en runbot
Copy link
Sponsor Member

Choose a reason for hiding this comment

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

Demonstration in

@max3903
Copy link
Sponsor Member

max3903 commented Nov 18, 2016

@damendieta Apart from cosmectic changes 👍

@coveralls
Copy link

Coverage Status

Coverage remained the same at 81.818% when pulling 79212e7 on damendieta:ote into 826f93d on OCA:9.0.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 81.818% when pulling 79212e7 on damendieta:ote into 826f93d on OCA:9.0.

@@ -0,0 +1,222 @@
"id","state_id:id","code","name"
"l10n_ec_ote_canton_0101","state_ec_01","0101","CUENCA"
Copy link

Choose a reason for hiding this comment

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

No need to use l10n_ec_ote_ on the l10n_ec_ote_canton_0101 id, only canton_0101 should work, same for the parish and state id's.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 81.818% when pulling 26f44df on damendieta:ote into 826f93d on OCA:9.0.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 81.818% when pulling 26f44df on damendieta:ote into 826f93d on OCA:9.0.

@max3903 max3903 added this to the 9.0 milestone Nov 18, 2016
<field name="model">res.partner</field>
<field name="inherit_id" ref="base.view_partner_form"/>
<field name="arch" type="xml">
<xpath expr="//field[@name='city']" position="replace">
Copy link

Choose a reason for hiding this comment

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

This xpath are dangerous, use the same as in res_partner view definition.

Copy link
Author

Choose a reason for hiding this comment

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

I think you meant company's view... fixed :D

@coveralls
Copy link

Coverage Status

Coverage remained the same at 81.818% when pulling eda82d3 on damendieta:ote into 826f93d on OCA:9.0.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 81.818% when pulling eda82d3 on damendieta:ote into 826f93d on OCA:9.0.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 81.818% when pulling 7a1482c on damendieta:ote into 826f93d on OCA:9.0.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 81.818% when pulling 7a1482c on damendieta:ote into 826f93d on OCA:9.0.

@damendieta damendieta closed this Nov 19, 2016
@damendieta damendieta reopened this Nov 19, 2016
@coveralls
Copy link

coveralls commented Nov 19, 2016

Coverage Status

Coverage remained the same at 81.818% when pulling 7a1482c on damendieta:ote into 826f93d on OCA:9.0.

@coodec coodec merged commit d04e87f into OCA:9.0 Nov 19, 2016
damendieta added a commit to damendieta/l10n-ecuador that referenced this pull request Nov 19, 2016
[NEW MODULE] New module with data for Ecuador geopolitical localization
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants