-
Notifications
You must be signed in to change notification settings - Fork 195
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: kind based nostr permissions #3054
Conversation
add hover states to badges
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.
Great progress 💯 Can't wait to finally use it myself.
src/extension/background-script/actions/nostr/signEventOrPrompt.ts
Outdated
Show resolved
Hide resolved
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.
NIP04 / NIP44 translations are currently missing, please just use "Decrypt / Encrypt data", no mention of NIP04 / NIP44 - users won't know what this is.
Can we possibly also merge them into one nostr/encrypt / nostr/decrypt permission? For a user I think it makes no difference which internal method is used to encrypt / decrypt data.
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.
tACK
Describe the changes you have made in this PR
kind based nostr permissions + overall permission refactoring in extension
Type of change
feat
: New feature (non-breaking change which adds functionality)How has this been tested?
try permisisons functionality
Checklist