[C-5129, C-5130] Add reply/mention push/email notifications #9998
Conversation
|
|
Preview this change https://demo.audius.co/c-5129-reply-notifications |
| ) | ||
| } | ||
|
|
||
| // If the user has devices to the notification to, proceed |
There was a problem hiding this comment.
| // If the user has devices to the notification to, proceed | |
| // If the user has devices to send the notification to, proceed |
| ) | ||
| } | ||
|
|
||
| // If the user has devices to the notification to, proceed |
There was a problem hiding this comment.
| // If the user has devices to the notification to, proceed | |
| // If the user has devices to send the notification to, proceed |
| comment_id: mention.comment_id, | ||
| user_id: mention.user_id, | ||
| created_at: new Date(Date.now()), | ||
| updated_at: new Date(Date.now()), |
There was a problem hiding this comment.
Curious - do we generally follow this pattern (set updated_at during creation), or do we leave it null until an update?
There was a problem hiding this comment.
it is set to created on initialization is what i gather from other models
Updates notification plugin to add mobile push and browser notifications. Also adds 'mentions' to identity-service records, and updates ui