Help wanted: fix missing translation strings

[epenet]

The problem

As a developper, it's very easy to forget adding all expected translation keys into strings.json
I am working on catching this within the regular test routines (see #127787)

If you would like to help, take a look at the resulting failures and see if you can add/fix:
https://github.com/home-assistant/core/pull/127787/checks

Tasks

Beta Give feedback

1. cloudflare config.error.unknown Add missing translation string in cloudflare #127906
2. crownstone config.error.unknown_error Fix incorrect translation key in crownstone #127913
3. emoncms options.error.failure
4. enphase_envoy config.abort.reconfigure_successful Add missing translation string in enphase envoy #128053
5. eq3btsmart config.error.invalid_mac_address Fix translation string in eq3btsmart #128319
6. fivem config.error.unknown Fix incorrect translation string in fivem #127907
7. gardena_bluetooth config.abort.cannot_connect Fix translation string in gardena_bluetooth #128387
8. generic `config.error
9. google config.abort.missing_credentials Fix translation string in google #128237
10. google_assistant_sdk config.abort.single_instance_allowed Fix translation string in google_assistant_sdk #128238
11. google_photos config.abort.reauth_successful Add missing translation string for re-auth flows #128055
12. google_tasks config.abort.reauth_successful Add missing translation string for re-auth flows #128055
13. group options.step.event.data.entities
14. guardian config.error.cannot_connect
15. hardkernel config.abort.single_instance_allowed Mark integrations as single_config_entry in manifest [system integrations] #128201
16. hassio config.abort.already_configured Use single_instance_allowed in hassio #128060
17. homeassistant_green config.abort.single_instance_allowed Mark integrations as single_config_entry in manifest [system integrations] #128201
18. homeassistant_yellow config.abort.single_instance_allowed Mark integrations as single_config_entry in manifest [system integrations] #128201
19. homewizard config.abort.unsupported_api_version Adjust homewizard translation strings #128423
20. homeworks config.abort.reconfigure_successful
21. hvv_departures options.error.cannot_connect
22. hvv_departures options.error.invalid_auth
23. hydrawise config.error.invalid_auth
24. hyperion options.abort.cannot_connect Fix translation string in hyperion #128384
25. iotty config.abort.missing_credentials Fix translation string in iotty #128385
26. isy994 config.abort.reauth_successful Add missing translation string for re-auth flows #128055
27. jvc_projector config.abort.reauth_successful Add missing translation string for re-auth flows #128055
28. knocki config.error.invalid_auth Fix translation string in knocki #128318
29. lamarzocco config.error.machine_not_found
30. landisgyr_heat_meter config.error.cannot_connect
31. lg_netcast config.abort.invalid_host
32. lifx config.abort.cannot_connect Fix translation string in lifx #128362
33. local_calendar config.abort.already_configured Add missing already_configured strings #128058
34. luftdaten config.abort.already_configured Add missing already_configured strings #128058
35. lyric config.abort.missing_credentials Fix translation string in lyric #128386
36. madvr config.abort.set_up_new_device
37. matter config.abort.cannot_connect Fix translation string in matter #128364
38. meater config.abort.reauth_successful Add missing translation string for re-auth flows #128055
39. melcloud config.abort.cannot_connect Fix translation string in melcloud #128363
40. melcloud config.abort.invalid_auth Fix translation string in melcloud #128363
41. met_eireann config.abort.already_configured Add missing already_configured strings #128058
42. microbees config.abort.reauth_successful Add missing translation string for re-auth flows #128055
43. motioneye config.abort.already_in_progress Fix translation string in motioneye #128403
44. nina config.abort.unknown Don't abort on unknown error in nina config flow #127908
45. nina options.abort.unknownDon't abort on unknown error in nina config flow #127908
46. ollama config.abort.download_failed
47. ondilo_ico config.abort.single_instance_allowed Mark integrations as single_config_entry in manifest [k-r] #128191
48. onewire `options.abort
49. otbr config.abort.already_configured Add missing translation string in otbr #127909
50. otbr config.abort.unknown Add missing translation string in otbr #127909
51. ovo_energy config.error.authorization_error
52. ovo_energy config.error.connection_error
53. permobil config.abort.unknown Add missing and fix incorrect translation string in permobil #127910
54. permobil config.step.region.data.region Add missing and fix incorrect translation string in permobil #127910
55. philips_js config.abort.pairing_failure
56. raspberry_pi config.abort.single_instance_allowed
57. risco config.abort.reauth_successful Add missing translation string for re-auth flows #128055
58. rova config.error.cannot_connect Fix translation string in rova #128402
59. rympro config.abort.reauth_successful Add missing translation string for re-auth flows #128055
60. smappee config.abort.already_configured Add missing already_configured strings #128058
61. spotify config.abort.missing_credentials
62. subaru config.abort.two_factor_request_failed
63. sunweg config.abort.already_configured Add missing already_configured strings #128058
64. surepetcare config.abort.reauth_successful Add missing translation string for re-auth flows #128055
65. tankerkoenig options.error.invalid_auth Fix translation string in tankerkoenig #128320
66. tesla_fleet config.abort.reauth_account_mismatch Add missing translation string in tesla_fleet #127915
67. tesla_fleet config.abort.reauth_successful Add missing translation string in tesla_fleet #127915
68. tessie config.abort.reauth_successful Add missing translation string for re-auth flows #128055
69. tile config.error.unknown Add missing translation string in tile #127911
70. tplink config.abort.cannot_connect Fix translation string in tplink #128352
71. tradfri config.error.invalid_security_code
72. triggercmd config.error.connection_errorFix incorrect error strings in triggercmd #128450
73. triggercmd config.error.invalid_token Fix incorrect error strings in triggercmd #128450
74. unifiprotect config.abort.reauth_successful Add missing translation string for re-auth flows #128055
75. utility_meter config.error.tariffs_not_unique
76. vilfo config.error.wrong_host
77. weatherflow config.error.no_device_found Fix translation string in weatherflow #128321
78. webmin `config.error Fix incorrect error strings in webmin #128448
79. whirlpool config.abort.reauth_successful Add missing translation string for re-auth flows #128055
80. withings config.abort.reauth_successful Add missing translation string for re-auth flows #128055
81. yamaha_musiccast config.error.cannot_connect Add missing translation string in yamaha_musiccast #127912
82. yamaha_musiccast config.error.unknown Add missing translation string in yamaha_musiccast #127912
83. yolink config.abort.missing_credentials Fix translation string in yolink #128353
84. youtube config.abort.wrong_account Add missing translation for youtube #128431

[mib1185]

These are all open ones (without a pending PR)

...

Edit: converted to task list in the description

[epenet]

we have some integrations (eq. generic and webmin, properly much more) which redirects possible connection errors direct into the errors dict to the async_show_form, those they can have unexpected or unforeseen error values, which can't be translated 🤔 Should we adjust the tests in this case, so they test only expected errors? Or do we've the possibility to work with placeholders also for errors in flow forms?

Might be worth confirming with a core member...
I think we normally use "unknown" and log the full details

[gjohansson-ST]

I think we normally use "unknown" and log the full details

Like that yes, we can only translate known errors e.g. there is no placeholders or such.

[epenet]

I've updated the task list to remove (at least temporarily) the data strings, and add the latest PRs from @j4n-e4t

[j4n-e4t]

@epenet Do you have a way to get all components that still have the ignore_translations rule in the config_flow tests? Should I go through them and file a PR removing them for the merged ones?
I found quite some components still having such rule in place, even if the translation is added correctly. Also my merged PRs from yesterday.

[epenet]

@epenet Do you have a way to get all components that still have the ignore_translations rule in the config_flow tests? Should I go through them and file a PR removing them for the merged ones? I found quite some components still having such rule in place, even if the translation is added correctly. Also my merged PRs from yesterday.

I've opened #128441 to remove them all
There was a gap in the previous detection logic