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

Extension: remove unused permission #8680

Merged
merged 1 commit into from
Nov 15, 2024
Merged

Extension: remove unused permission #8680

merged 1 commit into from
Nov 15, 2024

Conversation

PopDaph
Copy link
Contributor

@PopDaph PopDaph commented Nov 15, 2024

Description

We don't use activeTab.
I'm afraid it would be better for the submission to use this over tabs since we currently don't use other tab than the active one but since this is something we want to offer to users trying as is.

Risk

/

Deploy Plan

/

@PopDaph PopDaph requested a review from tdraier November 15, 2024 13:59
@PopDaph
Copy link
Contributor Author

PopDaph commented Nov 15, 2024

PMRR 🙏🏻

@PopDaph PopDaph merged commit d3f6e3d into main Nov 15, 2024
3 checks passed
@PopDaph PopDaph deleted the ext-remove-permission branch November 15, 2024 14:00
@tdraier
Copy link
Contributor

tdraier commented Nov 15, 2024

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants