Skip to content

Conversation

@legalsylvain
Copy link
Contributor

@legalsylvain legalsylvain commented Jan 4, 2026

[REF] web_view_leaflet_map: full refactor.
- rename folder to follow odoo and OCA conventions, regarding views definition
- isolate leaflet_map_view code
- isolate leaflet_map_controller code
- isolate leaflet_map_renderer code
- split xml file into renderer and controller part
- apply Odoo SA convention

[FIX] remove useless logs, and put a single one in debug level
- remove useless code
- remove obsolete directive: control-panel-bottom-right

[IMP] web_view_leaflet_map: introduce 'Create' Button

Before

image

After

image

@legalsylvain legalsylvain force-pushed the 18.0-IMP-web_view_leaflet_map branch 2 times, most recently from 1e276b3 to 472a6cc Compare January 5, 2026 10:40
@legalsylvain legalsylvain changed the title 18.0 imp web view leaflet map [18.0] web_view_leaflet_map : add 'Create' missing button Jan 5, 2026
@legalsylvain legalsylvain marked this pull request as ready for review January 5, 2026 10:51
- rename folder to follow odoo and OCA conventions, regarding views definition
- isolate leaflet_map_view code
- isolate leaflet_map_controller code
- isolate leaflet_map_renderer code
- split xml file into renderer and controller part
- apply Odoo SA convention

[FIX] remove useless logs, and put a single one in debug level
- remove useless code
- remove obsolete directive: control-panel-bottom-right

[IMP] web_view_leaflet_map: introduce 'Create' Button
@legalsylvain legalsylvain force-pushed the 18.0-IMP-web_view_leaflet_map branch from 472a6cc to d37bc89 Compare January 5, 2026 10:55
@legalsylvain
Copy link
Contributor Author

CC, recent contributors / reviewers : @mourad-ehm @thibaultrey, @tate11, @NL66278

Note : I investigated a little, regarding the possibility to add a search bar (Ref: #419 (comment)), but it requires a full refactoring, because pointers rendering are not done in a classic "odoo" way for the time being.

Also ideally, we should make a dynamic filter, to avoid to load records that are "out" of the displayed map. (for the time being, the map size will be adapted, depending on the records.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant