Skip to content

fix #2309#2310

Merged
slingamn merged 4 commits intoergochat:masterfrom
slingamn:issue2309
Dec 22, 2025
Merged

fix #2309#2310
slingamn merged 4 commits intoergochat:masterfrom
slingamn:issue2309

Conversation

@slingamn
Copy link
Copy Markdown
Member

If bob is monitoring alice, bob should get METADATA lines for alice even if bob doesn't have extended-monitor. The four caps that explicitly require extended-monitor are: away-notify, chghost, setname, account-notify.

If bob is monitoring alice, bob should get METADATA lines for alice
even if bob doesn't have extended-monitor. The four caps that explicitly
require extended-monitor are: away-notify, chghost, setname, account-notify.
@slingamn slingamn added this to the 2.18 milestone Dec 16, 2025
@slingamn
Copy link
Copy Markdown
Member Author

We might not even need to check extended-monitor --- we need to advertise it to tell clients that we support it, but there's no real harm in sending, e.g. CHGHOST to a client that requested chghost but not extended-monitor. If they're not tracking the target, they'll just ignore it.

@slingamn slingamn merged commit 3c4c5dd into ergochat:master Dec 22, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant