-
Notifications
You must be signed in to change notification settings - Fork 26
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
Import mozilla-mobile/fenix quarantine. #249
Import mozilla-mobile/fenix quarantine. #249
Conversation
* Rename shortcuts to search engine on search screen * Remove deprecated pref key * Re-add deprecated string X-Channel-Revision: [master] mozilla-mobile/android-components@1321b1d X-Channel-Converted-Revision: [master] mozilla-mobile/fenix@b7325a3 X-Channel-Revision: [master] mozilla-mobile/firefox-tv@e8ccd59 X-Channel-Revision: [main] MozillaReality/FirefoxReality@e52556c X-Channel-Revision: [master] mozilla-lockwise/lockwise-android@7243b05
@@ -1454,12 +1454,14 @@ | |||
<string name="top_sites_max_limit_confirmation_button">OK, Got It</string> | |||
|
|||
<!-- DEPRECATED STRINGS --> |
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.
@eliserichards What would be best here would be to just delete all those depecrated strings. Does that make sense to you as well?
thanks
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.
@BranescuMihai: I think Elise is out, would you mind removing all the deprecated strings so I can expose all the strings this weekend to localizers please? (so, the ones landed by Elise as well as the new one from 7fa41a2)
The rest is good to go!
ed5876c
to
86cf3f5
Compare
e9ea870
to
0d3e818
Compare
X-Channel-Revision: [master] mozilla-mobile/android-components@1321b1d X-Channel-Converted-Revision: [master] mozilla-mobile/fenix@bfc955c X-Channel-Revision: [master] mozilla-mobile/firefox-tv@e8ccd59 X-Channel-Revision: [main] MozillaReality/FirefoxReality@e52556c X-Channel-Revision: [master] mozilla-lockwise/lockwise-android@7243b05
X-Channel-Revision: [master] mozilla-mobile/android-components@1321b1d X-Channel-Converted-Revision: [master] mozilla-mobile/fenix@5d1aeb5 X-Channel-Revision: [master] mozilla-mobile/firefox-tv@e8ccd59 X-Channel-Revision: [main] MozillaReality/FirefoxReality@e52556c X-Channel-Revision: [master] mozilla-lockwise/lockwise-android@7243b05
X-Channel-Revision: [master] mozilla-mobile/android-components@1321b1d X-Channel-Converted-Revision: [master] mozilla-mobile/fenix@ff50dae X-Channel-Revision: [master] mozilla-mobile/firefox-tv@e8ccd59 X-Channel-Revision: [main] MozillaReality/FirefoxReality@e52556c X-Channel-Revision: [master] mozilla-lockwise/lockwise-android@7243b05
@@ -1454,12 +1454,14 @@ | |||
<string name="top_sites_max_limit_confirmation_button">OK, Got It</string> | |||
|
|||
<!-- DEPRECATED STRINGS --> |
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.
@BranescuMihai: I think Elise is out, would you mind removing all the deprecated strings so I can expose all the strings this weekend to localizers please? (so, the ones landed by Elise as well as the new one from 7fa41a2)
The rest is good to go!
7fa41a2
to
fdf9559
Compare
As requested here: mozilla-l10n/android-l10n#249 (comment)
d7c5547
to
3d64ae5
Compare
As requested here: mozilla-l10n#249 (comment) X-Channel-Revision: [master] mozilla-mobile/android-components@1321b1d X-Channel-Converted-Revision: [master] mozilla-mobile/fenix@3cb1acc X-Channel-Revision: [master] mozilla-mobile/firefox-tv@e8ccd59 X-Channel-Revision: [main] MozillaReality/FirefoxReality@e52556c X-Channel-Revision: [master] mozilla-lockwise/lockwise-android@7243b05
As requested here: mozilla-l10n/android-l10n#249 (comment)
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.
LGTM!
n/t