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.

Sign up for GitHub

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

Jump to bottom

Custom notification metadata [backend] #426

Closed
chris13524 opened this issue Mar 20, 2024 · 1 comment · May be fixed by #505
Closed

Custom notification metadata [backend] #426

chris13524 opened this issue Mar 20, 2024 · 1 comment · May be fixed by #505
Assignees
@chris13524
Labels
accepted

Comments

@chris13524
Copy link
Member

In addition to title, body, etc., allow the dapp to attach any type of custom "metadata" to a notification as a string. This metadata will be available in any SDK and has no defined structure or purpose.

This could be modeled after Stripe's Metadata: https://stripe.com/docs/api/metadata

For example:

{
  notification: {
    type: 'a1e53b95-18e5-4af8-9f03-9308ec87b687',
    title: 'The price of Ethereum has risen',
    body: 'The price of Ethereum has gone up 10%',
    url: 'https://app.example.com/',
    data: '{"field":"value"}'
  },
  accounts: ['eip155:1:0x9AfEaC202C837df470b5A145e0EfD6a574B21029']
}

Slack threads:
@chris13524 chris13524 self-assigned this Mar 20, 2024
@arein arein added the accepted label Mar 20, 2024
@linear Linear
Copy link

linear bot commented Mar 20, 2024

W3I-37 Custom notification metadata

@chris13524 chris13524 mentioned this issue Mar 20, 2024
Draft
@chris13524 chris13524 mentioned this issue Apr 17, 2024
Open
3 tasks
@chris13524 chris13524 changed the title Custom notification metadata Custom notification metadata [backend] Apr 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chris13524 chris13524

Labels
accepted
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: notification metadata WalletConnect/notify-server
2 participants
@arein @chris13524