-
-
Notifications
You must be signed in to change notification settings - Fork 30
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
Feature: Limit permissions for the extension to GitHub #102
Comments
@diivi this is not quite a good first issue until it provides the solution. https://opensauced.pizza/blog/good-first-issues-dont-exist |
@Anush008 can the authentication really function without having access to websites other than GitHub? I tried changing the host_permissions, and couldn't log in anymore. If so,
|
@diivi, we'll be needing Line 5 in 554bded
ai/src/utils/checkAuthentication.ts Lines 10 to 14 in 554bded
The getter above won't return a value without access to https://insights.opensauced.pizza/* .
|
Type of feature
🍕 Feature
Current behavior
Suggested solution
By default these settings is marked for "all sites"
Additional context
No response
Code of Conduct
Contributing Docs
The text was updated successfully, but these errors were encountered: