Skip to content

Conversation

@mourad-ehm
Copy link
Contributor

Migrate web_view_leaflet_map module from this pr #415

@tate11
Copy link

tate11 commented Oct 22, 2025

web.assets_web_dark.min.js:41 Uncaught Error: Error while loading "@web_view_leaflet_map/components/map-component/map_view.esm":
Error: Invalid object: 'type' is not valid
at ModuleLoader.startModule (web.assets_web_dark.min.js:41:227)
at ModuleLoader.startModules (web.assets_web_dark.min.js:40:57)
at ModuleLoader.addJob (web.assets_web_dark.min.js:5:39)
at ModuleLoader.define (web.assets_web_dark.min.js:10:121)
at web.assets_web_dark.min.js:22891:6
web.assets_web_dark.min.js:23 The following modules failed to load because of an error: Array(1)
reportErrors @ web.assets_web_dark.min.js:23
share-modal.js:1 Uncaught TypeError: Cannot read properties of null (reading 'addEventListener')
at share-modal.js:1:135

@thibaultrey thibaultrey force-pushed the 18.0-mig-web_view_leaflet_map branch from 75628c2 to ae08db6 Compare November 7, 2025 13:40
@mourad-ehm mourad-ehm changed the title WIP [18.0][MIG] web_view_leaflet_map: Migration to 18.0 [18.0][MIG] web_view_leaflet_map: Migration to 18.0 Nov 12, 2025
legalsylvain and others added 26 commits November 12, 2025 10:50
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/
This reverts commit a9570109
@mourad-ehm mourad-ehm force-pushed the 18.0-mig-web_view_leaflet_map branch from ae08db6 to a4c2500 Compare November 12, 2025 09:53
@mourad-ehm
Copy link
Contributor Author

Hi @tate11, thanks for your review. The bug was fixed. could you test ?

@legalsylvain
Copy link
Contributor

Hi @mourad-ehm. Thanks for porting this module. I know that generaly, there is a module per PR, but in that case, could you also migrate trivial web_view_leaflet_map_partner module, so we can test on runboat, more easely ?

thanks !

@mourad-ehm
Copy link
Contributor Author

Hi @legalsylvain, I have created this PR #420 for web_view_leaflet_map_partner.

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@mourad-ehm mourad-ehm requested a review from NL66278 November 17, 2025 11:00
@legalsylvain
Copy link
Contributor

Hi. tested on #425 / runboat.
works like a charm.
thanks for porting this module.

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-419-by-legalsylvain-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 370808a into OCA:18.0 Nov 17, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@bealdav bealdav deleted the 18.0-mig-web_view_leaflet_map branch November 17, 2025 11:44
@legalsylvain
Copy link
Contributor

Hi. I installed the module on odoo 18 instance, and it looks like there is a regression.
The search bar is not visible in the map view. (it was present in v16 and V12)
it is so not possible to filter...

image

Did you faced similar issue ?

@NL66278
Copy link
Contributor

NL66278 commented Nov 20, 2025

@legalsylvain Yes, I do not see a searchbar on 18.0. So same here.

@legalsylvain
Copy link
Contributor

@mourad-ehm. Could you take a look on that regression ?
Thanks !

@NL66278
Copy link
Contributor

NL66278 commented Nov 20, 2025

@legalsylvain @mourad-ehm I can get the map to show a searchbar with the attached patch. However it does not yet react to selected filters. So will need some more work, but I have not to much experience in owl how to do this, so maybe use this as a pointer to what might be needed?
searchbar.patch

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.

10 participants