-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Bloganuary: Replace hyphen with dash in Bloganuary nudge text #22213
Conversation
Generated by 🚫 dangerJS |
For the month of January, blogging prompts will come from Bloganuary - \ | ||
For the month of January, blogging prompts will come from Bloganuary — \ | ||
our community challenge to build a blogging habit for the new year. | ||
""", |
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.
Given the red background, it appears GitHub thinks this is a syntax error. The file compiles fine though, but I can change this to a single line if we'd like.
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.
Decided to just update it: 6155351.
📲 You can test the changes from this Pull Request in WordPress Alpha by scanning the QR code below to install the corresponding build.
|
📲 You can test the changes from this Pull Request in Jetpack Alpha by scanning the QR code below to install the corresponding build.
|
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.
Tested locally, works as expected.
@wargcm +1 for manually editing the .strings
as the hotfix automation does not download new strings from GlotPress because GlotPress holds the source of truth for the next upcoming version and as such there might be strings that were in the version an hotfix targets that are no longer in GlotPress, or new strings that are downloaded but unused.
Worth noting that:
- Until the
en.lproj
change reachestrunk
, the GlotPress entry will still hold the version with the hyphen - Many of the translators opted to remove the hyphen in their locale in favor of comma or semicolon. I found this interesting. I noticed this when
git grep "Bloguanary -" -- WordPress/Resources
returned only a couple of additional entries from theen-*
edited in the first version of this PR. I wonder if they picked up on the typo or simply felt their version better suited their locale 🤷♂️
One final note. I double checked that the dahs used here was the "em dash" not the "en dash". The syntactic difference matters.. 👌
Method used: Copied the character from the Merriam-Webster link above, and pasted it in the search window for this diff in Chrome, verified there were 5 matches, one per each of the files touched.
Just waiting for green CI to before merging... |
Ref: pcdRpT-4FE-p2#comment-7965
Description
Replaces the hyphen (-) with a dash (—) in the Bloganuary nudge text on the dashboard card. Matching Android's string change here: wordpress-mobile/WordPress-Android#19774
Screenshot
Testing
To test:
Regression Notes
Potential unintended areas of impact
N/A
What I did to test those areas of impact (or what existing automated tests I relied on)
N/A
What automated tests I added (or what prevented me from doing so)
N/A
PR submission checklist:
RELEASE-NOTES.txt
if necessary.UI Changes testing checklist: