Skip to content

Map layers title translations fixed#2487

Open
stefanocudini wants to merge 1 commit intoGeoNode:masterfrom
stefanocudini:map-layers-title-translations_fix
Open

Map layers title translations fixed#2487
stefanocudini wants to merge 1 commit intoGeoNode:masterfrom
stefanocudini:map-layers-title-translations_fix

Conversation

@stefanocudini
Copy link
Copy Markdown
Member

@stefanocudini stefanocudini commented Apr 3, 2026

add support for Multilanguage in Layers title at added new layer.

  • added 1 test for check action UPDATE LAYER on ADD_LAYER and title multilanguage value
  • include a patch for working in debugmode
Peek.2026-03-10.15-36.mp4

NOTE:
For testing require GeoNode/geonode#13985 at backend but it is backwards compatible (single language title for dataset)

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot 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

This pull request implements a feature to synchronize map layer titles with GeoNode's internationalized dataset titles. Key changes include adding i18n support to API calls, introducing a synchronization epic, and adding a new button to the Datasets Catalog plugin. Feedback indicates that the synchronization epic incorrectly attempts to access layer data from the action payload instead of the state, and the getDatasetsByName API function needs to be updated to fetch localized titles. Additionally, there are suggestions to remove hardcoded placeholder text from the UI and clean up redundant arguments in the Redux connection.

@allyoucanmap allyoucanmap requested review from anup39 and removed request for allyoucanmap April 9, 2026 07:49
@allyoucanmap allyoucanmap assigned anup39 and unassigned allyoucanmap Apr 9, 2026
@stefanocudini stefanocudini force-pushed the map-layers-title-translations_fix branch from 19e9a26 to f4bcb79 Compare April 14, 2026 15:40
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.

3 participants