-
Notifications
You must be signed in to change notification settings - Fork 13.1k
[NEW][Enterprise] Micro services #19000
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
Conversation
|
This pull request introduces 1 alert when merging 4a0f569 into 9daf8e4 - view on LGTM.com new alerts:
|
|
This pull request introduces 1 alert when merging b36bb69 into 61e4498 - view on LGTM.com new alerts:
|
|
This pull request introduces 1 alert when merging c345e65 into 3b06de0 - view on LGTM.com new alerts:
|
# Conflicts: # app/lib/server/startup/userDataStream.js # app/models/server/raw/index.ts # definition/IMessage.ts # definition/IRoom.ts # definition/ISubscription.ts # definition/IUser.ts
| } | ||
|
|
||
| export const msgStream = new MessageStream('room-messages'); | ||
| export const msgStream = notifications.streamRoomMessage; |
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.
this doesn't look to be used anywhere
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.
it is :( need to change that then
|
This pull request introduces 2 alerts when merging b453930 into 3995eec - view on LGTM.com new alerts:
|
|
This pull request introduces 1 alert when merging bd60936 into 3995eec - view on LGTM.com new alerts:
|
Proposed changes
Issue(s)
How to test or reproduce
Screenshots
Types of changes
Checklist
Changelog
Further comments