Skip to content

[16.0][IMP] fieldservice_geoengine: Updated display information for legends#1531

Open
anusriNPS wants to merge 1 commit into
OCA:16.0from
PyTech-SRL:16.0-enhance-legend
Open

[16.0][IMP] fieldservice_geoengine: Updated display information for legends#1531
anusriNPS wants to merge 1 commit into
OCA:16.0from
PyTech-SRL:16.0-enhance-legend

Conversation

@anusriNPS
Copy link
Copy Markdown
Contributor

Based on #456 base_geoengine PR, updating patched functions to display legend information only for records with coordinate details.

@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hi @wolfhall, @max3903,
some modules you are maintaining are being modified, check this out!

@OCA-git-bot OCA-git-bot added mod:fieldservice_geoengine Module fieldservice_geoengine series:16.0 labels Apr 30, 2026
@anusriNPS anusriNPS force-pushed the 16.0-enhance-legend branch 2 times, most recently from 1d93788 to 484eaee Compare May 4, 2026 14:57
Copy link
Copy Markdown
Contributor

@quirino95 quirino95 left a comment

Choose a reason for hiding this comment

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

Code and functional review, LGTM!

@anusriNPS anusriNPS force-pushed the 16.0-enhance-legend branch 2 times, most recently from 29ec42d to f540c39 Compare May 6, 2026 14:33
…egends

   Based on OCA#456 base_geoengine PR, updating patched functions to
display legend information only for records with coordinate details.
@anusriNPS anusriNPS force-pushed the 16.0-enhance-legend branch from f540c39 to a982c96 Compare May 6, 2026 14:58
Copy link
Copy Markdown
Contributor

@HekkiMelody HekkiMelody left a comment

Choose a reason for hiding this comment

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

Code review, LGTM

@OCA-git-bot
Copy link
Copy Markdown
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). 🤖

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.

4 participants