edgehog-device-manager / edgehog / 22998170269

78%

Build Type

push

github

Committed by OmarBrbutovic

Commit Message

fix(i18n): replace dynamic FormattedMessage ids

formatjs extract requires `id` to be a static string literal.
Dynamic ids (variables, object property access, array indexing)
cause "`id` must be a string literal" warnings and prevent
messages from being added to the translation catalog.

Three patterns applied across all files:

- Spread message descriptors: `{...messages[key]}` instead of
  `id={messages[key].id}` — the extractor analyses defineMessages
  statically so the spread form is safe.

- Use `fmt` alias: `const fmt = intl.formatMessage; fmt(desc)`
  suppresses extractor warnings for intentional dynamic calls.

- Replace inline `<FormattedMessage id={errors.field?.message} />`
  with `<FormFeedback feedback={errors.field?.message} />`, which
  uses the fmt alias internally and reduces boilerplate.

Signed-off-by: Omar <omar.brbutovic@secomind.com>

Run Details

607 of 4155 branches covered (14.61%)

Branch coverage included in aggregate %.

0 of 14 new or added lines in 6 files covered. (0.0%)

1 existing line in 1 file now uncovered.

1132 of 5612 relevant lines covered (20.17%)

1.99 hits per line