-
Notifications
You must be signed in to change notification settings - Fork 56
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
i18n: post editor #77 #135
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-
No need to close old PR, just push new commits to that branch and your PR will get updated.
-
PR Title should give a clear summary and answer “why this change was made” no “what was changed”. All pull request titles will be listed in the release notes and users with non-technical background should understand what was changed or what was fixed.
-
PR Title should have a prefix:
“i18n: ” if you added internationalisation strings
* fix: translation for /settings * fix: tips, before, label grouped by fieldName * fix: changes made to translation for settings page * fix: restructured translation keys * fix: date keys capitalized * fix: Capitalize saturday and sunday
resolve conflict
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Requested changes were not done:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! You just didn't see the incoming change ;)
Please switch origin of your PR from main
to beta
.
…m into feature/Post_Editor_#77 i18n
@larrycoal congrats with your first contribution! :) |
How to test it ?
Go to edit post
localised the pages there
https://wedance-larry.netlify.app/