Skip to content

Feat/df 1242: Enable Welsh translation for 'form unavailable' page - #455

Merged
jbarnsley10 merged 10 commits into
mainfrom
feat/df-1242-form-unavailable-welsh
Aug 7, 2026
Merged

Feat/df 1242: Enable Welsh translation for 'form unavailable' page#455
jbarnsley10 merged 10 commits into
mainfrom
feat/df-1242-form-unavailable-welsh

Conversation

@jbarnsley10

@jbarnsley10 jbarnsley10 commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Proposed change

Enable Welsh translation for 'form unavailable' page

  • also fixes location instructions translations when no JS enabled

Jira ticket: DF-1242

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 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.

LGTM Jez - just that 1 query from me.

instructionSummary: t('components.locationFieldBase.howToFind'),
instructionText:
tComponent(component.def, 'instructionText') ||
component.options.instructionText,

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.

Why the fallback here? Will tComponent not already fallback to the English version?

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.

Yes, it's overkill. I'll remove

@jbarnsley10
jbarnsley10 enabled auto-merge August 7, 2026 12:32
@sonarqubecloud

sonarqubecloud Bot commented Aug 7, 2026

Copy link
Copy Markdown

@jbarnsley10
jbarnsley10 merged commit f40efad into main Aug 7, 2026
23 checks passed
@jbarnsley10
jbarnsley10 deleted the feat/df-1242-form-unavailable-welsh branch August 7, 2026 12:33
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