Update How We Refer to Fakespot
Categories
(Firefox for Android Graveyard :: Shopping, task, P1)
Tracking
(firefox119 disabled, firefox120 verified, firefox121 verified)
People
(Reporter: wrocklin, Assigned: 007)
References
Details
(Whiteboard: [fxdroid] [fakespot-android-mvp])
Attachments
(3 files)
-
We are moving to Title Case for the feature name. So, no more "review checker" or "Review checker". Always "Review Checker". Please refer to the string inventory to see which strings need to be updated with this change. Filter the last updated column for October 18." Link to string inventory if you need it --> https://docs.google.com/spreadsheets/d/1FQUU2pqhKkAXRHNbmK9dhObwvevWoR3N6BOh3fu2skE/edit?usp=sharing
-
We only need to say "Fakespot by Mozilla" on the first reference. On subsequent references can say just "Fakespot". There are two strings we can update from "Fakespot by Mozilla" to just "Fakespot". One is at the bottom of the opt-in sidebar, above the Try It button. The other is at the end of the "How we determine..." card. These updates are in the string inventory. Filter the last updated column for October 18." https://docs.google.com/document/d/1RjOhF6uA_W7Bdxg6u2P7CB5gbeMgcI08F0SXx2RAOBs/edit#heading=h.ym9ledgoymmj
Updated•2 years ago
|
Updated•2 years ago
|
Updated•2 years ago
|
| Assignee | ||
Updated•2 years ago
|
| Assignee | ||
Updated•2 years ago
|
Comment 1•2 years ago
|
||
Comment 2•2 years ago
|
||
Authored by https://github.com/MozillaNoah
https://github.com/mozilla-mobile/firefox-android/commit/20e13a636354c7c46b413cad325d1c7e85c69272
[main] Bug 1859906 - Update two usages of "Fakespot by Mozilla" to just be "Fakespot"
Comment 3•2 years ago
|
||
| Assignee | ||
Comment 4•2 years ago
|
||
Comment on attachment 9361277 [details] [review]
[mozilla-mobile/firefox-android] Bug 1859906 - Update two usages of "Fakespot by Mozilla" to just be "Fakespot" (backport #4290) (#4305)
Beta/Release Uplift Approval Request
- User impact if declined: The copy for two mentions of Fakespot will be incorrect.
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): This is a copy change involving two strings
- String changes made/needed: Two strings were changed
- Is Android affected?: Yes
Comment 5•2 years ago
|
||
:Noah please see comment in backport https://github.com/mozilla-mobile/firefox-android/pull/4305#issuecomment-1789364001
Comment 6•2 years ago
|
||
Verified on the Fenix Nightly 121.0a1 from 11/1 with Oppo Find N2 Flip (Android 13).
Updated•2 years ago
|
| Assignee | ||
Comment 7•2 years ago
|
||
Thanks for the callout, Dianna; I'm surprised the backport doesn't tag me on the uplift review at least for tracking purposes. I'm pretty sure this is a hard product requirement though, but I am verifying this today.
Comment 8•2 years ago
|
||
Comment 9•2 years ago
|
||
Authored by https://github.com/MozillaNoah
https://github.com/mozilla-mobile/firefox-android/commit/5f573d61e70e7338da6521ef4e2ffecfb23090d3
[releases_v120] Bug 1859906 - Update two usages of "Fakespot by Mozilla" to just be "Fakespot"
Comment 10•2 years ago
|
||
Verified as fixed on the Firefox Beta 120.0b7 with Motorola G9 Plus (Android 11) and Sony Xperia (Android 6.0.1).
Updated•1 year ago
|
Description
•