RocketChat / RocketChat/Rocket.Chat
no client in this repo subscribes to notify-logged/user-status; if mobile and the
Open
Nobody has claimed this yet.
todo
type: feature
- Dominant language
- TypeScript
- Stars
- 46.1k
- Forks
- 13.9k
- Avg merge
- 3d 3h
- Merged PRs (30d)
- 130
Description
SDK don't either, this check and the restricted mirror behind it can be dropped
📝 Found in apps/meteor/server/modules/listeners/listeners.module.ts#L210
Commit: b89a8d411ef65f6931a5fd1cd057740bc00cd9ba
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at apps/meteor/server/modules/listeners/listeners.module.ts around line 210 and trace the notify-logged/user-status subscription and its restricted mirror. Verify whether any web, mobile, or SDK client subscribes to this event. Done means the check and mirror are removed only if no supported client depends on them, with relevant tests or checks passing.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100