Skip to content

DF-1226: Add Welsh language translations to the clientside maps - #456

Merged
davidjamesstone merged 10 commits into
mainfrom
feature/DF-1226-map-translations
Aug 7, 2026
Merged

DF-1226: Add Welsh language translations to the clientside maps#456
davidjamesstone merged 10 commits into
mainfrom
feature/DF-1226-map-translations

Conversation

@davidjamesstone

@davidjamesstone davidjamesstone commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Proposed change

Maps plugin: Add Welsh translation capability

Jira ticket: https://eaflood.atlassian.net/browse/DF-1226

Type of change

  • Bug fix
  • New feature
  • Breaking change
  • Misc. (documentation, build updates, etc)

Checklist

  • You have executed this code locally and it performs as expected.
  • You have added tests to verify your code works.
  • You have added code comments and JSDoc, where appropriate.
  • There is no commented-out code.
  • You have added developer docs in README.md and docs/* (where appropriate, e.g. new features).
  • The tests are passing (npm run test).
  • The linting checks are passing (npm run lint).
  • The code has been formatted (npm run format).

@davidjamesstone
davidjamesstone force-pushed the feature/DF-1226-map-translations branch from 2845f6d to de0a612 Compare August 6, 2026 11:55
@davidjamesstone
davidjamesstone marked this pull request as ready for review August 7, 2026 11:43
@sonarqubecloud

sonarqubecloud Bot commented Aug 7, 2026

Copy link
Copy Markdown

disabled = false,
readonly = false
readonly = false,
lang = DEFAULT_LANG

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just a general point about passing an optional 'lang' param ... my preference would be to have the 'lang' required in all cases apart from the top level call (where it might default to en-GB say). That way you can be sure you've not forgotten to pass a 'lang' into an inner method i.e. the linter would pick that up.
But if you have sufficient test coverage, it should be fine.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I kept this in because this function is also used by forms-designer (to generate the "map review" page) and it doesn't have a language switch. I could make it mandatory but we'd have to then pass in en-GB from designer

@jbarnsley10 jbarnsley10 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Good work. Just my observation about the optional 'lang' param but I'll leave that up to you to decide what you think is best.

@davidjamesstone
davidjamesstone merged commit a030548 into main Aug 7, 2026
26 checks passed
@davidjamesstone
davidjamesstone deleted the feature/DF-1226-map-translations branch August 7, 2026 14:51
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.

2 participants