search

OCA / geospatial

Odoo and GIS
http://oca.github.io/geospatial/index.html
GNU Affero General Public License v3.0
187 stars 281 forks source link

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

Closed reichie020212 closed 7 months ago

dreispt commented 7 months ago

@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 commented 7 months ago

@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 https://github.com/OCA/geospatial/pull/357/commits/ef6c9009252925a580247988cdb7f6d10e410699, I squash them using git rebase to reduce the commit noise

dreispt commented 7 months ago

@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 commented 7 months ago

@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.

reichie020212 commented 7 months ago

@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 commented 7 months ago

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 commented 7 months ago

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.