-
Notifications
You must be signed in to change notification settings - Fork 800
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(experimental): add voteNotifications
websocket method
#1647
feat(experimental): add voteNotifications
websocket method
#1647
Conversation
9c52694
to
28e7608
Compare
81f4945
to
509e538
Compare
28e7608
to
78155c3
Compare
509e538
to
d7d0d75
Compare
78155c3
to
0a85d2c
Compare
d7d0d75
to
8718140
Compare
0a85d2c
to
0b53e8c
Compare
8718140
to
50ee179
Compare
@@ -22,6 +22,7 @@ export function getAllowedNumericKeypathsForNotification(): AllowedNumericKeypat | |||
if (!memoizedNotificationKeypaths) { | |||
memoizedNotificationKeypaths = { | |||
slotsUpdatesNotifications: [['timestamp']], | |||
voteNotifications: [['timestamp']], |
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.
Typed as i64
on the server, so I don't think you should cast it, unless I'm missing something?
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 guess I got confused since the UnixTimestamp
is a number
not bigint
, so I thought we always have to cast it
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'm also confused why these numbers are typed the way they are. Casting to number seems reasonable!
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.
0b53e8c
to
11216d8
Compare
50ee179
to
5b4bccc
Compare
11216d8
to
0819243
Compare
5b4bccc
to
f1b9656
Compare
f1b9656
to
b12c149
Compare
0819243
to
4d0b8a6
Compare
b12c149
to
dae4387
Compare
4e29d96
to
b4673f6
Compare
853763b
to
24a09be
Compare
🎉 This PR is included in version 1.81.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Because there has been no activity on this PR for 14 days since it was merged, it has been automatically locked. Please open a new issue if it requires a follow up. |
No description provided.