-
-
Notifications
You must be signed in to change notification settings - Fork 852
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
[13.0][MIG] base_location: Migration to 13.0 #785
Conversation
… when company country is filled.
…ssues with module partner_address_street3
…some views. Added two columns in respective tree views.
…ooks * Added Icon. * Improve module description and extracted to README.rst. * Pass country instead of country_id for advance comparisons. * Allow to transform city name. * Some code style. * Do not remove all entries of a country, but only not found. * Include hooks for transforming some things. * Include spanish translation.
Brussels Belgium city
…y, state and country in order to allow search using '%' from the m2o widget
* Headers shortened * Move cities management to settings
* [IMP] base_location: Add lat & long to `better.zip` * Add latitude and longitude columns to `better.zip` * [IMP] base_location_geonames_import: Add lat/long * Add support for latitude & longitude to genomes importer
Incredibly not included in Odoo core.
Currently translated at 66.7% (24 of 36 strings) Translation: partner-contact-11.0/partner-contact-11.0-base_location Translate-URL: https://translation.odoo-community.org/projects/partner-contact-11-0/partner-contact-11-0-base_location/es/
Currently translated at 100,0% (36 of 36 strings) Translation: partner-contact-11.0/partner-contact-11.0-base_location Translate-URL: https://translation.odoo-community.org/projects/partner-contact-11-0/partner-contact-11-0-base_location/de/
This module has now been refactored to be more consistent with what base_address_city offers to the location management. Added dependency to contacts so that I could change the menu location for cities / zip management. Now, every res.city record has a relation One2many to res.city.zip (old res.better.zip). This way, every zip has a realted city too. Zips can be searched through city code, zip or city name (same as before). Modified tests and deleted not needed tests. Added sql contraints so that zips and cities are unique within it's country / state / city.
Steps: - Open company form - Set 'city completion' field Get 'The state of the partner My Company differs from that in location X' Disabling _check_zip while writing 'zip' fields from company, as incompatible with the sequence of write operations. Automatic test is added too.
Currently translated at 57.6% (19 of 33 strings) Translation: partner-contact-12.0/partner-contact-12.0-base_location Translate-URL: https://translation.odoo-community.org/projects/partner-contact-12-0/partner-contact-12-0-base_location/es/
Previous script required to have cities populated for working, but that's not usual thing, as people in v11 may choose to not import them, and for older databases they even weren't that option. With this improve script, now we populate city table for ZIP entries without city, so the rest of the queries are properly executed in any case.
/ocabot merge |
Hey, thanks for contributing! Proceeding to merge this for you. |
@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 13.0-ocabot-merge-pr-785-by-pedrobaeza-bump-no. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
/ocabot merge |
On my way to merge this fine PR! |
@pedrobaeza The merge process could not be finalized, because command
|
@moylop260 why 13.0 is not admitted in pylint-odoo yet? I thought OCA/pylint-odoo#249 was for that. |
pylint is supporting 13.0 The lint travis build is green I think it is another tool |
/ocabot merge |
Hey, thanks for contributing! Proceeding to merge this for you. |
@pedrobaeza The merge process could not be finalized, because command
|
@pedrobaeza @sbidoul is looking into this now |
Great, thanks |
/ocabot merge |
What a great day to merge this nice PR. Let's do it! |
@sbidoul I created https://odoo-community.org/web#id=847934&action=468&active_id=2&model=project.task&view_type=form&menu_id=748 for all the things to do on a new version, so if there's something for the bot, please indicate it there |
@pedrobaeza thanks for the initiative. Updated. |
@sbidoul The merge process could not be finalized, because command
|
/ocabot merge |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at e29c9a2. Thanks a lot for contributing to OCA. ❤️ |
cc @Tecnativa TT19707