-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
P3A for android #6176
Comments
Added @anthonypkeane to drive mobile UX for notification and onboarding for existing and new users -- should parallel desktop, but will have to be custom-designed. |
Let's use this issue to capture the Android specific work needed to get P3A working on Android. #6688 captures the UI changes needed |
@AlexeyBarabash any reason why the issue is marked as QA/No? Shouldn't we test the changes for P3A? The associated PR has a proper test plan mentioned in it |
at that time it required modified apk, which would hard to provide to QA team, and also the task wasn't complete without UI part, so it was marked QA/No With new changes at #12723 it requires a new test plan |
To launch P3A on android we have to properly inform users about this feature.
Until then, P3A service is disabled in the code for android (see
BraveBrowserProcessImpl::brave_p3a_service()
usages.The text was updated successfully, but these errors were encountered: