Skip to content

Conversation

@christoph-jerolimov
Copy link
Contributor

@christoph-jerolimov christoph-jerolimov commented Oct 14, 2022

In the OpenShift Console, we missed some translations in our resource bundle. Also, the English ResourceBundle missed some translations from this package.

In this case, the UI doesn't show anything because getResource returns an empty string if the translation isn't found in the ResourceBundle.

In this case, getResource should return the input resource string instead so that the UI shows something. If just the translation is missing it shows at least an English version.

@netlify
Copy link

netlify bot commented Oct 14, 2022

Deploy Preview for quickstarts ready!

Name Link
🔨 Latest commit 34ed121
🔍 Latest deploy log https://app.netlify.com/sites/quickstarts/deploys/6349df6b823f6b0009258e48
😎 Deploy Preview https://deploy-preview-194--quickstarts.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@christoph-jerolimov christoph-jerolimov changed the title Fallback to resource string (English version) instead of showing nothing getResource doesn't shown anything if translation string is missing in the ResourceBundle (incl. English) Oct 14, 2022
@christoph-jerolimov
Copy link
Contributor Author

/cc @jschuler

@jschuler
Copy link
Collaborator

Thank you @jerolimov !

@jschuler jschuler merged commit b0bd26b into patternfly:main Oct 31, 2022
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