Skip to content

Add /watch chat command#38152

Open
jotkauser wants to merge 2 commits into
ppy:masterfrom
jotkauser:feat/watch-command
Open

Add /watch chat command#38152
jotkauser wants to merge 2 commits into
ppy:masterfrom
jotkauser:feat/watch-command

Conversation

@jotkauser

Copy link
Copy Markdown

I've noticed that there's no /watch chat command, but this command exists in stable, so i decided to add it here into lazer.

Comment thread osu.Game/Online/Chat/WatchCommand.cs Fixed

@bdach bdach left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aside from the code quality inspection that needs to be fixed:

var request = new GetUserRequest(username);
request.Success += user =>
{
performer.PerformFromScreen(s => s.Push(new SoloSpectatorScreen(user)));

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On stable, the command also fails if the user is not online. Technically this could be achieved via MetadataClient via something like

IDisposable? token = null;
if (!IsWatchingUserPresence)
   token = BeginWatchingUserPresence();
var status = GetPresence(userId)?.Status;
token?.Dispose();

if (status != Online) Fail();

I'm just not sure that it should be because this will work fine without it, by showing the spectator splash screen with the user card on it showing that the user is offline.

@peppy your opinion on how to proceed here please.

@bdach bdach added area:overlay-chat subjective PRs with subjective changes which have not been discussed prior. Need team consensus to proceed. labels Jun 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:overlay-chat size/M subjective PRs with subjective changes which have not been discussed prior. Need team consensus to proceed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants