Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(client): use translations json for button text #54737

Merged

Conversation

a2937
Copy link
Member

@a2937 a2937 commented May 11, 2024

Checklist:

Related to #54715
Closes #54791

While that PR was good, I think things could been better if the translations file is used.

@a2937 a2937 requested a review from a team as a code owner May 11, 2024 01:18
@github-actions github-actions bot added platform: learn UI side of the client application that needs familiarity with React, Gatsby etc. scope: i18n language translation/internationalization. Often combined with language type label labels May 11, 2024
@naomi-lgbt naomi-lgbt changed the title chore(ui): use translations json for button text fix(client): use translations json for button text May 14, 2024
@huyenltnguyen huyenltnguyen merged commit 010515f into freeCodeCamp:main May 15, 2024
22 checks passed
@a2937 a2937 deleted the chore(ui)/migrate-submit-button-trans branch May 15, 2024 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform: learn UI side of the client application that needs familiarity with React, Gatsby etc. scope: i18n language translation/internationalization. Often combined with language type label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace hidden-xs class with an isMobile check
3 participants