There is a comma missing after the word "heard" within a launcher string used for a migration notification
The Bug:
There is a comma missing after the word "heard" within a launcher string used for a migration notification.
This is a very minor issue however, this word should have a comma present after it in order for the string to make grammatical sense. A comma should always be used to connect an independent and dependent clause, where the dependent clause comes before the independent clause. Not using a comma under these circumstances would be grammatically incorrect.
Below is a table I've constructed that highlights this error. You can find information about what the current and expected string is and would be, some context regarding it, and a link to the appropriate string on crowdin.
| Current String | Expected String | String Context | String URL on Crowdin |
|---|---|---|---|
| As you %1$smight have heard%2$s we’re migrating all Mojang accounts to Microsoft accounts to increase account security and player safety. | As you %1$smight have heard%2$s, we’re migrating all Mojang accounts to Microsoft accounts to increase account security and player safety. | #: ..\..\apps\launcher\src\components\MigrationNotifications\index.tsx:116 | https://crowdin.com/translate/minecraft-launcher/9420/en-engb#5273454 |
Steps to Reproduce:
- Locate the affected string as listed above.
- Take note as to whether or not there is a comma missing after the word "heard" within a launcher string used for a migration notification.
Observed Behavior:
There is a comma missing after the word "heard".
Expected Behavior:
A comma would be present after the word "heard" so that the string makes grammatical sense.
2022-04-09, 12:12 PM
2022-06-22, 05:41 PM
2022-06-22, 05:41 PM
0
1
762912
-