Skip to content
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

Remove verbose state events from the timeline #2216

Closed
Tracked by #2254
jmartinesp opened this issue Jan 12, 2024 · 0 comments · Fixed by #2225
Closed
Tracked by #2254

Remove verbose state events from the timeline #2216

jmartinesp opened this issue Jan 12, 2024 · 0 comments · Fixed by #2225
Assignees
Labels
T-Task Refactoring, enabling or disabling functionality, other engineering tasks

Comments

@jmartinesp
Copy link
Member

jmartinesp commented Jan 12, 2024

When creating a room, we can see at least 7 state events being added to the timeline:

  • Who created the room (m.room.create).
  • The same person joining the room (m.room.member with membership join).
  • The room power levels being set (m.room.power_levels).
  • The room join rules being set up (m.room.join_rules).
  • The room history visibility being set up (m.room.history_visibility).
  • The room guest access being set up (m.room.guest_access).
  • If encryption is enabled or disabled (m.room.encryption).

Plus any invites.

From those, we only want to display:

  • Who created the room.
  • Who joined the room.
  • If encryption is enabled.
  • The invites.

So we need to filter out the rest of the events from the timeline. Following iOS' implementation, this removal should be done globally in the timeline, not just for the few first items.

@jmartinesp jmartinesp added the T-Task Refactoring, enabling or disabling functionality, other engineering tasks label Jan 12, 2024
@jmartinesp jmartinesp self-assigned this Jan 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Task Refactoring, enabling or disabling functionality, other engineering tasks
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant