Skip to content

Conversation

@maxim-productengine
Copy link
Contributor

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.

@maxim-productengine maxim-productengine changed the title Nearby chat voice moderation [Draft] Nearby chat voice moderation Nov 7, 2025
LLSD body;
body["operand"] = operand;
body["agent_id"] = agent_id;
body["moderator_id"] = gAgent.getID();
Copy link
Contributor

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.

Copy link
Contributor Author

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();
Copy link
Contributor

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)
Copy link
Contributor

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)?

Copy link
Contributor Author

@maxim-productengine maxim-productengine Nov 13, 2025

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants