-
Couldn't load subscription status.
- Fork 2.4k
Bug 1954524 data collection permissions key in manifest #28276
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
base: main
Are you sure you want to change the base?
Bug 1954524 data collection permissions key in manifest #28276
Conversation
|
Tip: Review these changes grouped by change (recommended for most PRs), or grouped by feature (for large PRs). |
|
@willdurand I noticed that the change was released in 139 ,but the page on Extension Workshop states that it's available for 140 (and 142 for Android). Which version should be followed here and on the MDN release note? |
|
@willdurand, please correct me if I'm wrong, but I believe AMO's linter enforces the 140 requirement. While technically the code exists in Firefox 139 does support it, the official publishing flow doesn't. |
|
The desktop UI was finalized in 140 so let's go with that. |
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.
r+wc
| }, | ||
| "edge": "mirror", | ||
| "firefox": { | ||
| "version_added": "140" |
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.
| "version_added": "140" | |
| "version_added": "140", | |
| "notes": "Supports `required` and `optional`." |
| "version_added": "140" | ||
| }, | ||
| "firefox_android": { | ||
| "version_added": "142" |
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.
| "version_added": "142" | |
| "version_added": "142", | |
| "notes": "Supports `required` and `optional`." |
|
@willdurand I've added details of the optional and required properties to the documentation. Therefore, in line with what we normally document in the BCD, I don't think we need the note about them here. |
Summary
Add details of the
data_collection_permissionsproperty ofgeckoto thebrowser_specific_settingsmanifest key. This feature was implemented in Bug 1954524 Add support for data collection permissions in the manifest.Related issues
Fixes #28211
Related MDM documentation included in PR TBC