-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Enable Translate feature by default #25289
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
Comments
@atuchin-m could you provide a test plan for QA to use if manual QA is required here? if not, we should have this labeled as |
@LaurenWags IMHO we need only sanity check that the feature is enabled despite the Griffin config. |
@atuchin-m is the following sufficient?
|
Yes, that's enough. |
perfect, thanks for confirming @atuchin-m! |
Verified with
Verified steps from #25289 (comment). Confirmed on a clean profile (without restart, so no griffin seed pulled), that Brave's translate service was offered, not the Google extension:
|
The feature is nearly to be rolled to 100% of users.
brave/brave-variations#355
The feature should be enabled by default in the code and the griffin experiment should be finished.
The text was updated successfully, but these errors were encountered: