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.

Sign up for GitHub

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

Jump to bottom

[17.0][MIG] base_geoengine: Migration to 17.0 #357

Closed
wants to merge 455 commits into from
Closed

[17.0][MIG] base_geoengine: Migration to 17.0 #357

wants to merge 455 commits into from

Conversation

reichie020212
Copy link

No description provided.

@reichie020212 reichie020212 mentioned this pull request Jan 30, 2024
Open
4 tasks
@dreispt
Copy link
Member

dreispt commented Jan 30, 2024

@reichie020212 I believe you missed the procedure to bring the fill commit history.
See https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-17.0

@reichie020212
Copy link
Author

@reichie020212 I believe you missed the procedure to bring the fill commit history. See https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-17.0

@dreispt full commit history is in this commit ef6c900, I squash them using git rebase to reduce the commit noise

@dreispt
Copy link
Member

dreispt commented Jan 31, 2024

@reichie020212 Got it. We want the full history, as it provides information on the changes dates, authors and reasons.
Only redundant commits, if any, need to be squashed.

@reichie020212
Copy link
Author

@reichie020212 Got it. We want the full history, as it provides information on the changes dates, authors and reasons. Only redundant commits, if any, need to be squashed.

@dreispt Ahh, I see. Noted on this. I'll update the PR.

yvaucher and others added 25 commits January 31, 2024 16:17
@yvaucher @reichie020212
By default use OSM
34cec6e
@yvaucher @reichie020212
Fix path of openerp for doc build
3c2df6e
@yvaucher @reichie020212
Remove google from selection as it is currently broken
ba4a1a9
@yvaucher @reichie020212
Add geo_contains operator to geo_search method.
6feb1c5 
geo_contains calls St_Contains SQL search.

Plus fix search to set SRID to avoid mixin srid issue.
@yvaucher @reichie020212
Refactor WMTS to be able to add a generic config
35b9264 
And extract Swisstopo to move it in a new module
@yvaucher @reichie020212
Add a button on map widget to clear geometry from the map
e40e5bc
@yvaucher @reichie020212
Fix _setup_fields signature due to OCA/OCB@87d160a
db2ecb9 
Otherwise partial value is interpreted as context and it will fails on
Odoo loading
@yvaucher @reichie020212
Bump version
11358f1
@yvaucher @reichie020212
Adapt test to mandatory partial attribute in _setup_fields
cfd7a5c
@yvaucher @reichie020212
Fix srid value read which was unset in field properties
8762862
@yvaucher @reichie020212
Add parameters to define extends and projection
1874956
@yvaucher @reichie020212
Adapt tests
286bb9b
@yvaucher @reichie020212
Bump version
cc5c5d6
@yvaucher @reichie020212
Target only the right tab where the map is to add listener.
f95bda0
@fredj @reichie020212
Ensure we call add_tab_listener only once
10a0fcb
@yvaucher @reichie020212
Address the right container when drawing the widget map
debe085
@yvaucher @reichie020212
Fix widget initialized too early in second tab thus having bad size
0d0bd1d
@yvaucher @reichie020212
Fix offset issues on the map by reseting offset when scroll bar is used
180297c
@yvaucher @reichie020212
Don't zoom when setting or changing a geometry
d67af56
@yvaucher @reichie020212
Add an option to set a default zoom to avoid showing a map that you a…
16360ef 
…lways need to zoom out to get useful informations
@yvaucher @reichie020212
No key argument in js, set an optional argument instead with default …
5e6da74 
…value
@yvaucher @reichie020212
Add an action on map info box to open the selected object in form view
e1e1365
@yvaucher @reichie020212
Update Roadmap
8b19e73
@yvaucher @reichie020212
Bump version
c9460e6
@yvaucher @reichie020212
Fix z-index of Openlayers map container to ensure it doesn't hide sea…
580b56f 
…rch drop down list
lmignon and others added 14 commits January 31, 2024 16:17
@lmignon @reichie020212
[FIX] base_geoengine: Fix documentation format
3ffac10
@reichie020212
[UPD] Update base_geoengine.pot
9a2b656
@OCA-git-bot @reichie020212
[UPD] README.rst
647cc87
@weblate @reichie020212
Update translation files
73630fd 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: geospatial-16.0/geospatial-16.0-base_geoengine
Translate-URL: https://translation.odoo-community.org/projects/geospatial-16-0/geospatial-16-0-base_geoengine/
@Ivorra78 @reichie020212
Added translation using Weblate (Spanish)
afe75a6
@Ivorra78 @reichie020212
Translated using Weblate (Spanish)
48635f3 
Currently translated at 100.0% (163 of 163 strings)

Translation: geospatial-16.0/geospatial-16.0-base_geoengine
Translate-URL: https://translation.odoo-community.org/projects/geospatial-16-0/geospatial-16-0-base_geoengine/es/
@OCA-git-bot @reichie020212
[UPD] README.rst
61185f1
@weblate @reichie020212
Update translation files
5a8a639 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: geospatial-16.0/geospatial-16.0-base_geoengine
Translate-URL: https://translation.odoo-community.org/projects/geospatial-16-0/geospatial-16-0-base_geoengine/
@jakobkrabbe @reichie020212
Added translation using Weblate (Swedish)
4a6ce3c
@jakobkrabbe @reichie020212
Translated using Weblate (Swedish)
68133de 
Currently translated at 77.9% (127 of 163 strings)

Translation: geospatial-16.0/geospatial-16.0-base_geoengine
Translate-URL: https://translation.odoo-community.org/projects/geospatial-16-0/geospatial-16-0-base_geoengine/sv/
@mymage @reichie020212
Added translation using Weblate (Italian)
767e808
@mymage @reichie020212
Translated using Weblate (Italian)
37b07a7 
Currently translated at 100.0% (163 of 163 strings)

Translation: geospatial-16.0/geospatial-16.0-base_geoengine
Translate-URL: https://translation.odoo-community.org/projects/geospatial-16-0/geospatial-16-0-base_geoengine/it/
@reichie020212
[IMP] base_geoengine: pre-commit auto fixes
ca91f65
@reichie020212
[MIG] base_geoengine: Migration to 17.0
62abfac
@reichie020212 reichie020212 force-pushed the 17.0-mig-base_geoengine branch from 455e42f to 62abfac Compare January 31, 2024 08:25
@reichie020212
Copy link
Author

@reichie020212 Got it. We want the full history, as it provides information on the changes dates, authors and reasons. Only redundant commits, if any, need to be squashed.

@dreispt Ahh, I see. Noted on this. I'll update the PR.

@dreispt commit history is now updated.

@reichie020212
Copy link
Author

I encountered a javascript error when trying to access the geoengine view mode, I tried fixing it but to no avail. Can someone help me on this? Thank you.

@reichie020212
Copy link
Author

in the latest commit, I fixed some issue and I think the only issue remaining is the error when trying to click the geo fields in the geoengine view mode.

@miquelalzanillas miquelalzanillas mentioned this pull request May 2, 2024
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.