Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Bug 1796146 - Add UI for the Cookie Banner preferences #27561

Merged
merged 1 commit into from
Nov 15, 2022

Conversation

owlishDeveloper
Copy link
Contributor

@owlishDeveloper owlishDeveloper commented Oct 27, 2022

image

Pull Request checklist

  • Tests: This PR includes thorough tests or an explanation of why it does not
  • Screenshots: This PR includes screenshots or GIFs of the changes made or an explanation of why it does not
  • Accessibility: The code in this PR follows accessibility best practices or does not include any user facing features. In addition, it includes a screenshot of a successful accessibility scan to ensure no new defects are added to the product.

QA

  • QA Needed

To download an APK when reviewing a PR (after all CI tasks finished running):

  1. Click on Checks at the top of the PR page.
  2. Click on the firefoxci-taskcluster group on the left to expand all tasks.
  3. Click on the build-debug task.
  4. Click on View task in Taskcluster in the new DETAILS section.
  5. The APK links should be on the right side of the screen, named for each CPU architecture.

GitHub Automation

Used by GitHub Actions.

@owlishDeveloper owlishDeveloper force-pushed the cookie-banner branch 2 times, most recently from 6b9011c to c739a63 Compare October 28, 2022 19:39
@owlishDeveloper owlishDeveloper marked this pull request as ready for review October 28, 2022 19:41
@owlishDeveloper owlishDeveloper requested review from a team as code owners October 28, 2022 19:41
@owlishDeveloper owlishDeveloper added the needs:review PRs that need to be reviewed label Oct 28, 2022
@owlishDeveloper owlishDeveloper changed the title [WIP][DO NOT MERGE] Add cookie banner handling settings Bug 1796146 - Add UI for the Cookie Banner preferences Oct 28, 2022
@owlishDeveloper owlishDeveloper added 🙅 waiting Issues that are blocked or has dependencies that are not ready and removed 🙅 waiting Issues that are blocked or has dependencies that are not ready labels Oct 28, 2022
@owlishDeveloper
Copy link
Contributor Author

owlishDeveloper commented Oct 28, 2022

Waiting on https://bugzilla.mozilla.org/show_bug.cgi?id=1797142 (otherwise only the settings will be changed by this UI, the functionality itself will not be working)

@Amejia481
Copy link
Contributor

Request for data collection review form

All questions are mandatory. You must receive review from a data steward peer on your responses to these questions before shipping new data collection.

  1. What questions will you answer with this data?
  • How users interact with the cookie banners handling feature.
  1. Why does Mozilla need to answer these questions? Are there benefits for users? Do we need this information to address product or business requirements?
  • This data will help us understand how the cookie banners handling feature is used.
  1. What alternative methods did you consider to answer these questions? Why were they not sufficient?
  • There are no other alternatives.

  1. Can current instrumentation answer these questions?
  • No.

  1. List all proposed measurements and indicate the category of data collection for each measurement, using the Firefox data collection categories found on the Mozilla wiki.

    Note that the data steward reviewing your request will characterize your data collection based on the highest (and most sensitive) category.
Measurement Description Data Collection Category Tracking Bug #
When the cookie banners handling setting page is visited Category 2 - interaction data https://bugzilla.mozilla.org/show_bug.cgi?id=1796146
Which setting is selected Category 2 - interaction data https://bugzilla.mozilla.org/show_bug.cgi?id=1796146
  1. Please provide a link to the documentation for this data collection which describes the ultimate data set in a public, complete, and accurate way.
  1. How long will this data be collected? Choose one of the following:
  • Until version 118. Renewal then may be decided.
  1. What populations will you measure?
  • All channels, all locales, all countries
 that the feature is active for.
  1. If this data collection is default on, what is the opt-out mechanism for users?
  • Default Glean SDK opt-out mechanism.

  1. Please provide a general description of how you will analyze this data.
  • Glean and Amplitude.

  1. Where do you intend to share the results of your analysis?
  • 
Only on Glean, Amplitude, and with mobile teams.

  1. Is there a third-party tool (i.e. not Telemetry) that you are proposing to use for this data collection?
  • No third-party tools.

@Amejia481 Amejia481 added the needs:data-review PR is awaiting a data review label Oct 31, 2022
@mergify
Copy link
Contributor

mergify bot commented Nov 1, 2022

This pull request has conflicts when rebasing. Could you fix it @owlishDeveloper? 🙏

@Amejia481
Copy link
Contributor

Amejia481 commented Nov 1, 2022

Looks like https://bugzilla.mozilla.org/show_bug.cgi?id=1797142 already landed 🎉

@Amejia481 Amejia481 force-pushed the cookie-banner branch 2 times, most recently from 4b3377d to f77b5f3 Compare November 10, 2022 17:01
@rocketsroger
Copy link
Contributor

Request for data collection review form

All questions are mandatory. You must receive review from a data steward peer on your responses to these questions before shipping new data collection.

1. What questions will you answer with this data?


* How users interact with the cookie banners handling feature.


2. Why does Mozilla need to answer these questions? Are there benefits for users? Do we need this information to address product or business requirements?


* This data will help us understand how the cookie banners handling feature is used.


3. What alternative methods did you consider to answer these questions? Why were they not sufficient?


* There are no other alternatives.


4. Can current instrumentation answer these questions?


* No.


5. List all proposed measurements and indicate the category of data collection for each measurement, using the Firefox data collection categories found on the Mozilla wiki.
   
   Note that the data steward reviewing your request will characterize your data collection based on the highest (and most sensitive) category.

Measurement Description Data Collection Category Tracking Bug #
When the cookie banners handling setting page is visited Category 2 - interaction data https://bugzilla.mozilla.org/show_bug.cgi?id=1796146
Which setting is selected Category 2 - interaction data https://bugzilla.mozilla.org/show_bug.cgi?id=1796146

6. Please provide a link to the documentation for this data collection which describes the ultimate data set in a public, complete, and accurate way.


* [https://dictionary.telemetry.mozilla.org/apps/fenix
  ](https://dictionary.telemetry.mozilla.org/apps/fenix%E2%80%A8)


7. How long will this data be collected? Choose one of the following:


* Until version 118. Renewal then may be decided.


8. What populations will you measure?


* All channels, all locales, all countries
   that the feature is active for.


9. If this data collection is default on, what is the opt-out mechanism for users?


* Default Glean SDK opt-out mechanism.


10. Please provide a general description of how you will analyze this data.


* Glean and Amplitude.


11. Where do you intend to share the results of your analysis?


* Only on Glean, Amplitude, and with mobile teams.


12. Is there a third-party tool (i.e. not Telemetry) that you are proposing to use for this data collection?


* No third-party tools.

Data Review

  1. Is there or will there be documentation that describes the schema for the ultimate data set in a public, complete, and accurate way?

Yes, through the metrics.yaml file and the Glean Dictionary

  1. Is there a control mechanism that allows the user to turn the data collection on and off?

Yes, through the "Send Usage Data" preference in the application settings

  1. If the request is for permanent data collection, is there someone who will monitor the data over time?

N/A, collection set to end or be renewed by version 118

  1. Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?

Category 2, Interaction data

  1. Is the data collection request for default-on or default-off?

default-on

  1. Does the instrumentation include the addition of any new identifiers?

No

  1. Is the data collection covered by the existing Firefox privacy notice?

Yes

  1. Does the data collection use a third-party collection tool?

No

Result

data-review+

@rocketsroger rocketsroger removed the needs:data-review PR is awaiting a data review label Nov 10, 2022
.experimenter.yaml Outdated Show resolved Hide resolved
nimbus.fml.yaml Outdated Show resolved Hide resolved
@Amejia481 Amejia481 force-pushed the cookie-banner branch 2 times, most recently from 5c25b1f to 5d09219 Compare November 11, 2022 03:57
@Amejia481
Copy link
Contributor

In Fenix we don't support "Reject cookies if possible REJECT_ALL(1)" like in Focus ?

Yeah, based on the mockups it looks like Fenix will only have on setting REJECT_OR_ACCEPT_ALL(2).
image

@Amejia481
Copy link
Contributor

If I unchecked "Reduce cookie banner," cookieBannerHandlingMode should be disabled, but when I go to a site that has a cookie banner, I don't see it . Only after I remove the app from the background and enter again do I see it.

Thanks for catching this, I think missed adding the reload for the current session.

@Amejia481
Copy link
Contributor

Thanks for all the helpful comments @iorgamgabriel!
I updated the PR with your suggetions!

app/metrics.yaml Outdated Show resolved Hide resolved
@Amejia481 Amejia481 removed the 🙅 waiting Issues that are blocked or has dependencies that are not ready label Nov 14, 2022
@iorgamgabriel
Copy link
Contributor

Looks good to me! 👍

@Amejia481 Amejia481 requested review from iorgamgabriel and removed request for a team November 14, 2022 16:01
@Amejia481 Amejia481 added the pr:needs-landing PRs that are ready to land [Will be merged by Mergify] label Nov 15, 2022
@mergify mergify bot merged commit 8589aec into mozilla-mobile:main Nov 15, 2022
@sv-amocirean
Copy link

Hello,
This was verified by the mobile QA Team. See more details on Bugzilla.
Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Feature:CookieBannerHandling needs:review PRs that need to be reviewed pr:needs-landing PRs that are ready to land [Will be merged by Mergify]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants