-
Notifications
You must be signed in to change notification settings - Fork 95
[Draft] Nearby chat voice moderation #4943
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
base: develop
Are you sure you want to change the base?
Conversation
…chat moderation might work
…ial code for muting indivudual / everyone via the capability
…allum/voice-moderation-2
…allum/voice-moderation-2
…add stub code for info request
| LLSD body; | ||
| body["operand"] = operand; | ||
| body["agent_id"] = agent_id; | ||
| body["moderator_id"] = gAgent.getID(); |
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.
There is no need to send the moderator_id: remove it. The server ignores that field and instead grabs the agent_id directly from the connection context.
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.
Will do.
|
|
||
| LLSD body; | ||
| body["operand"] = operand; | ||
| body["moderator_id"] = gAgent.getID(); |
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.
Remove the moderator_id field.
| bool LLFloaterIMContainer::isNearbyChatSpeakerSelected() | ||
| { | ||
| LLFolderViewItem *selectedItem = mConversationsRoot->getCurSelectedItem(); | ||
| if (NULL == selectedItem) |
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.
Has the viewer switched yet to using nullptr (c++-ism) instead of NULL (c-ism)?
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.
In this case it was probably just copy-paste from another place. It's more common just to do if (!selectedItem) check.
Speaking of NULL vs nullptr - there we no switch, but it makes sense to use nullptr at least in the new code.
New menu options for Nearby chat moderation are added.
Also new notification will be displayed after being muted/unmuted in Nearby chat or clicking 'voice' button while being muted.