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

Consider letting Snaps expose settings that show up alongside the MetaMask settings #2874

Open
FrederikBolding opened this issue Nov 4, 2024 · 6 comments · May be fixed by #2911
Open

Consider letting Snaps expose settings that show up alongside the MetaMask settings #2874

FrederikBolding opened this issue Nov 4, 2024 · 6 comments · May be fixed by #2911
Assignees

Comments

@FrederikBolding
Copy link
Member

FrederikBolding commented Nov 4, 2024

Consider letting Snaps expose settings that show up alongside the MetaMask settings.

Possibly let them expose an entire dynamic UI as a section in the settings view.

Consider how this experience differs between preinstalled and regular Snaps.

More details TBD.

  • Preinstalled Snaps have own category
  • Regular Snaps go in own settings page
@eriknson
Copy link
Member

Should we scope this to be for pre-installed snaps only initially for the v1? And then introduce it to anyone as a v2

@Montoya
Copy link
Collaborator

Montoya commented Nov 13, 2024

Yes, scope for pre-installed only for v1

@ajeetd
Copy link

ajeetd commented Nov 13, 2024

This ticket will cover: Preinstalled Snaps have own category

In future rather Regular Snaps may follow the same model.

@GuillaumeRx GuillaumeRx linked a pull request Nov 27, 2024 that will close this issue
@danroc
Copy link
Contributor

danroc commented Nov 28, 2024

We should consider adding the ability to mark a setting as privacy-related, allowing it to be included in the Privacy section of the settings.

@eriknson
Copy link
Member

@danroc I agree, we're planning to introduce the ability for snaps to introduce settings within our native categories (like the privacy one) in parallell to the settings redesign being shipped by Q1

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 a pull request may close this issue.

6 participants