-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
Refactor: Add chat message to Postgres when user status changes #18980
Refactor: Add chat message to Postgres when user status changes #18980
Conversation
bbb-common-message/src/main/scala/org/bigbluebutton/common2/msgs/GroupChatMsg.scala
Outdated
Show resolved
Hide resolved
akka-bbb-apps/src/main/scala/org/bigbluebutton/core/apps/users/ChangeUserAwayReqMsgHdlr.scala
Outdated
Show resolved
Hide resolved
akka-bbb-apps/src/main/scala/org/bigbluebutton/core/apps/users/ChangeUserAwayReqMsgHdlr.scala
Show resolved
Hide resolved
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.
Please ensure the message is written using internationalization . There are two messages app.chat.away
and app.chat.notAway
that were created for this purpose.
...l5/imports/ui/components/chat/chat-graphql/chat-message-list/page/chat-message/component.tsx
Outdated
Show resolved
Hide resolved
akka-bbb-apps/src/main/scala/org/bigbluebutton/core/apps/users/ChangeUserAwayReqMsgHdlr.scala
Outdated
Show resolved
Hide resolved
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.
lockSettingsProps.disablePublicChat
validation is wrong.
Kudos, SonarCloud Quality Gate passed! |
Automated tests Summary🚨 Test workflow has failedClick here to check the action test reports |
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.
lgtm!
f9f419c
into
bigbluebutton:v3.0.x-release
What does this PR do?
Inserts a chat message from the system into Postgres when a user changes their away status.
Motivation
When a user's away status changes a message should be sent in chat to notify other users of this change.