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

[17.0][MIG] geoengine_base_geolocalize: Migration to 17.0 #382

Merged
merged 27 commits into from
Oct 29, 2024

Conversation

peluko00
Copy link

@peluko00 peluko00 commented Oct 28, 2024

lmignon and others added 27 commits October 28, 2024 08:29
The modul add a new field 'geo_point' computed from partner_latitue and partner_longitude added by base_geolocalize. The geo_point is rendered in a map on the partner form
[FIX]
* _get_geo_path method: fix condition
* fix name of class bulk_encoder
* add a tracker into README.rst
* rename test class
Squashed commit of the following:

commit 527e215
Author: Lindsay <[email protected]>
Date:   Tue Jun 2 14:47:52 2020 +0200

    fixing typo

commit dd9c3ec
Author: Lindsay <[email protected]>
Date:   Tue Jun 2 14:42:47 2020 +0200

    fixing pylint travis

commit 7899f9b
Author: Lindsay <[email protected]>
Date:   Tue Jun 2 14:05:25 2020 +0200

    fix travis flake8 linting

commit 039e867
Author: Lindsay <[email protected]>
Date:   Tue Jun 2 13:54:07 2020 +0200

    fix travis pylint

commit fcfb09e
Author: Lindsay <[email protected]>
Date:   Tue Jun 2 13:45:09 2020 +0200

    fixing travis tests

commit ff2b39e
Author: Lindsay <[email protected]>
Date:   Tue Jun 2 12:07:45 2020 +0200

    fix import

commit cdf43dc
Author: Lindsay <[email protected]>
Date:   Tue Jun 2 11:47:21 2020 +0200

    add responses to travis for testing

commit 79da0ee
Author: Lindsay <[email protected]>
Date:   Tue Jun 2 11:40:55 2020 +0200

    disable pylint warning W0622

commit d6eab49
Author: Lindsay <[email protected]>
Date:   Tue Jun 2 10:29:58 2020 +0200

    formatting for linters
Currently translated at 100.0% (2 of 2 strings)

Translation: geospatial-16.0/geospatial-16.0-geoengine_base_geolocalize
Translate-URL: https://translation.odoo-community.org/projects/geospatial-16-0/geospatial-16-0-geoengine_base_geolocalize/it/
@peluko00 peluko00 force-pushed the 17.0-mig-geoengine_base_geolocalize branch from 51f92a7 to ea234e9 Compare October 28, 2024 07:35
Copy link

@mpascuall mpascuall left a comment

Choose a reason for hiding this comment

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

LGTM, tested in runboat

Copy link

@ppyczko ppyczko left a comment

Choose a reason for hiding this comment

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

LGTM

@peluko00
Copy link
Author

Could you review @yvaucher please. Thanks!

@yvaucher
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 17.0-ocabot-merge-pr-382-by-yvaucher-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit c79bdc2 into OCA:17.0 Oct 29, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 4a6ec7c. Thanks a lot for contributing to OCA. ❤️

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.