-
Notifications
You must be signed in to change notification settings - Fork 44
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
[#918] Consume Airy Events from new websocket in the UI #988
Conversation
@@ -109,6 +110,7 @@ private void startStream() { | |||
|
|||
// produce unread count metadata | |||
messageStream.selectKey((messageId, message) -> message.getConversationId()) | |||
.filter((conversationId, message) -> isFromContact(message)) |
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.
Fixes a bug wherein unread counts where increased for messages sent from Airy
26db359
to
c124aee
Compare
@@ -16,7 +16,7 @@ | |||
|
|||
"rules": { | |||
"react/prop-types": 0, | |||
"@typescript-eslint/no-unused-vars": ["error", {"varsIgnorePattern": "_"}], |
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.
We should always provide a reason when changing the rules: This is very useful when you want to remove properties from an object by destructuring. I.e.
const {name, ...dataWithOutName} = contact;
You don't need the name here, but need to state it explicitly
6768dd3
to
1db2eab
Compare
1db2eab
to
14eb5ec
Compare
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.
Really awesome refactoring ❤️ . The typescript stuff is 👍
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.
resolves #918