-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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: add mark message action as seen dto #3084
Conversation
NV-1180 Swagger is missing the body of /messages/actions/:type
the swagger annotations is missing the body of the request inside our API reference: https://docs.novu.co/api/mark-message-action-as-seen/ The missing is the body with |
@IsOptional() | ||
// eslint-disable-next-line @typescript-eslint/no-explicit-any | ||
payload: any; |
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.
Considering that payload is saved as Mixed type in the database, I think we can safely replace the any for a Record<string, unknown>
and thus only allow JavaScript object structures.
description: 'Message action payload', | ||
}) | ||
@IsOptional() | ||
// eslint-disable-next-line @typescript-eslint/no-explicit-any |
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.
I think this can be removed now
What change does this PR introduce?
Add
MarkMessageActionAsSeenDto
Why was this change needed?
Fixes NV-1180
Other information (Screenshots)