Skip to content
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

feat: add more extensions manifest key options #39599

Merged
merged 1 commit into from Aug 28, 2023
Merged

feat: add more extensions manifest key options #39599

merged 1 commit into from Aug 28, 2023

Conversation

codebytere
Copy link
Member

@codebytere codebytere commented Aug 21, 2023

Description of Change

This PR adds support for several more commonly used manifest.json extension key properties:

  • author - an optional manifest key that takes an object with an "email" key
  • host_permissions - host permissions are specified as match patterns, and each pattern identifies a group of URLs for which the extension is requesting extra privileges
  • short_name - a short name for the extension, where if given, will be used in contexts where the name field is too long.

Checklist

Release Notes

Notes: Added support for several more extensions manifest keys including host_permissions, author, and short_name

@codebytere codebytere added semver/minor backwards-compatible functionality target/25-x-y PR should also be added to the "25-x-y" branch. target/26-x-y PR should also be added to the "26-x-y" branch. target/27-x-y PR should also be added to the "27-x-y" branch. labels Aug 21, 2023
@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Aug 21, 2023
Base automatically changed from fix-privileged-access-tabs to main August 22, 2023 12:57
@codebytere codebytere added semver/minor backwards-compatible functionality and removed semver/minor backwards-compatible functionality labels Aug 22, 2023
Copy link
Member

@erickzhao erickzhao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

API LGTM

Copy link
Contributor

@jkleinsc jkleinsc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

API LGTM

@jkleinsc jkleinsc merged commit f369b14 into main Aug 28, 2023
19 checks passed
@jkleinsc jkleinsc deleted the manifest-keys branch August 28, 2023 14:02
@release-clerk
Copy link

release-clerk bot commented Aug 28, 2023

Release Notes Persisted

Added support for several more extensions manifest keys including host_permissions, author, and short_name

@trop
Copy link
Contributor

trop bot commented Aug 28, 2023

I was unable to backport this PR to "25-x-y" cleanly;
you will need to perform this backport manually.

@trop trop bot added needs-manual-bp/25-x-y and removed target/25-x-y PR should also be added to the "25-x-y" branch. labels Aug 28, 2023
@trop
Copy link
Contributor

trop bot commented Aug 28, 2023

I have automatically backported this PR to "27-x-y", please check out #39672

@trop trop bot removed the target/27-x-y PR should also be added to the "27-x-y" branch. label Aug 28, 2023
@trop
Copy link
Contributor

trop bot commented Aug 28, 2023

I have automatically backported this PR to "26-x-y", please check out #39673

@trop trop bot added in-flight/26-x-y merged/27-x-y PR was merged to the "27-x-y" branch. merged/26-x-y PR was merged to the "26-x-y" branch. and removed target/26-x-y PR should also be added to the "26-x-y" branch. in-flight/27-x-y in-flight/26-x-y labels Aug 28, 2023
MrHuangJser pushed a commit to MrHuangJser/electron that referenced this pull request Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api-review/approved ✅ merged/26-x-y PR was merged to the "26-x-y" branch. merged/27-x-y PR was merged to the "27-x-y" branch. semver/minor backwards-compatible functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants