Skip to content

Conversation

@cesarportas
Copy link
Contributor

This PR migrates the module web_view_leaflet_map to the target version, including a full refactor to OWL framework.

legalsylvain and others added 30 commits October 1, 2025 15:43
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: geospatial-16.0/geospatial-16.0-web_view_leaflet_map
Translate-URL: https://translation.odoo-community.org/projects/geospatial-16-0/geospatial-16-0-web_view_leaflet_map/
Currently translated at 100.0% (7 of 7 strings)

Translation: geospatial-16.0/geospatial-16.0-web_view_leaflet_map
Translate-URL: https://translation.odoo-community.org/projects/geospatial-16-0/geospatial-16-0-web_view_leaflet_map/it/
…aflet_lib, to be used by other modules, like 'web_widget_map'. (see : OCA/web#2953)
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: geospatial-16.0/geospatial-16.0-web_view_leaflet_map
Translate-URL: https://translation.odoo-community.org/projects/geospatial-16-0/geospatial-16-0-web_view_leaflet_map/
Currently translated at 100.0% (2 of 2 strings)

Translation: geospatial-16.0/geospatial-16.0-web_view_leaflet_map_partner
Translate-URL: https://translation.odoo-community.org/projects/geospatial-16-0/geospatial-16-0-web_view_leaflet_map_partner/it/
This reverts commit a9570109
Copy link

@LoxoLueiro LoxoLueiro left a comment

Choose a reason for hiding this comment

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

LGTM

@VizMan0616
Copy link

Will this feature be merge into 17.0?

@pedrobaeza
Copy link
Member

/ocabot migration web_view_leaflet_map

@legalsylvain is it OK for you?

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Jan 7, 2026
@OCA-git-bot OCA-git-bot mentioned this pull request Jan 7, 2026
5 tasks
Copy link
Contributor

@legalsylvain legalsylvain left a comment

Choose a reason for hiding this comment

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

Hi. yes, V18 is working on top of this PR V17. So 👍

note: @cesarportas : I re-refactored recently this module, if you want to cherry pick. (not a blocking point, as your PR is not at the HEAD branch).

See :
#434

Note 2 : if you have any idea to fix the "no search bar problem", please ping me.

In the meantime

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 17.0-ocabot-merge-pr-415-by-legalsylvain-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Jan 7, 2026
Signed-off-by legalsylvain
@OCA-git-bot
Copy link
Contributor

@legalsylvain your merge command was aborted due to failed check(s), which you can inspect on this commit of 17.0-ocabot-merge-pr-415-by-legalsylvain-bump-nobump.

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.

@legalsylvain
Copy link
Contributor

hi @cesarportas : could you rebase, and take a look on the CI ? thanks !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants