Skip to content

feat(ui/i18n): mark more missing strings for translation#39

Open
nadjaheitmann wants to merge 1 commit into
mainfrom
mark_strings_for_translations
Open

feat(ui/i18n): mark more missing strings for translation#39
nadjaheitmann wants to merge 1 commit into
mainfrom
mark_strings_for_translations

Conversation

@nadjaheitmann
Copy link
Copy Markdown
Contributor

Generated-By Mistral Vibe AI

(This PR may not contain all the strings, some will be added with #35)

@sbernhard
Copy link
Copy Markdown
Member

Actually, I would prefer to have "__" as "translate" instead of "_". See https://github.com/theforeman/foreman/blob/develop/webpack/assets/javascripts/react_app/components/HostDetails/ActionsBar/actions.js#L4

AFAIK, this is currently not the case for this project.

That said, I would vote to merge these.

@nadjaheitmann
Copy link
Copy Markdown
Contributor Author

Actually, I would prefer to have "__" as "translate" instead of "_". See https://github.com/theforeman/foreman/blob/develop/webpack/assets/javascripts/react_app/components/HostDetails/ActionsBar/actions.js#L4

AFAIK, this is currently not the case for this project.

I had the same thought and I agree. But we need to work with the current project 'settings' as you said.

That said, I would vote to merge these.

Please approve and let's merge it.

@nadjaheitmann nadjaheitmann force-pushed the mark_strings_for_translations branch from 9ef6cf9 to 51e19f3 Compare April 8, 2026 09:52
@nadjaheitmann nadjaheitmann force-pushed the mark_strings_for_translations branch 3 times, most recently from 079252e to ca45631 Compare April 20, 2026 11:47
@nadjaheitmann nadjaheitmann force-pushed the mark_strings_for_translations branch from ca45631 to 9119f82 Compare April 20, 2026 13:06
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