Skip to content

Do not show membership/profile/topic events in public rooms#33192

Draft
bxdxnn wants to merge 1 commit intoelement-hq:developfrom
bxdxnn:feature/no-member-events-flood
Draft

Do not show membership/profile/topic events in public rooms#33192
bxdxnn wants to merge 1 commit intoelement-hq:developfrom
bxdxnn:feature/no-member-events-flood

Conversation

@bxdxnn
Copy link
Copy Markdown

@bxdxnn bxdxnn commented Apr 17, 2026

See element-hq/element-x-android#6360

Checklist

@github-actions github-actions Bot added the Z-Community-PR Issue is solved by a community member's PR label Apr 17, 2026
@bxdxnn bxdxnn marked this pull request as ready for review April 17, 2026 20:01
@bxdxnn bxdxnn requested a review from a team as a code owner April 17, 2026 20:01
@bxdxnn bxdxnn requested review from dbkr and t3chguy April 17, 2026 20:01
@t3chguy t3chguy requested a review from a team April 17, 2026 20:42
@t3chguy
Copy link
Copy Markdown
Member

t3chguy commented Apr 17, 2026

Please write tests before seeking review

@bxdxnn bxdxnn force-pushed the feature/no-member-events-flood branch from 92c71ca to 590238c Compare April 18, 2026 15:13
@t3chguy
Copy link
Copy Markdown
Member

t3chguy commented Apr 20, 2026

Copy link
Copy Markdown
Member

@t3chguy t3chguy left a comment

Choose a reason for hiding this comment

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

Dismissing code review until Product reviews it

@bxdxnn
Copy link
Copy Markdown
Author

bxdxnn commented Apr 20, 2026

@mxandreas
Copy link
Copy Markdown
Contributor

As I already commented on the Android PR, this topic need comprehensive handling not local patches for specific use cases or pain: element-hq/element-x-android#6360 (comment)

From product/design perspective we can't accept this as is.

@bxdxnn bxdxnn marked this pull request as draft April 20, 2026 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Z-Community-PR Issue is solved by a community member's PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants