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

Rename RoomListHeader into LegacyRoomListHeader #29308

Merged
merged 2 commits into from
Feb 19, 2025

Conversation

florianduros
Copy link
Member

@florianduros florianduros commented Feb 18, 2025

In order to avoid a conflict on css classes during the work in https://github.com/element-hq/wat-internal/issues/204, the room list header is renamed as LegacyRoomListHeader.

@florianduros florianduros added the T-Task Tasks for the team like planning label Feb 18, 2025
@florianduros florianduros force-pushed the florianduros/new-room-list/legacy-room-list-header branch 3 times, most recently from ebf430c to e84e96d Compare February 18, 2025 16:13
@florianduros florianduros force-pushed the florianduros/new-room-list/legacy-room-list-header branch from e84e96d to 737cd4f Compare February 18, 2025 16:18
@florianduros florianduros added the X-Run-All-Tests When applied to PRs, it'll run the full gamut of end-to-end tests on the PR label Feb 18, 2025
@florianduros florianduros marked this pull request as ready for review February 19, 2025 09:36
@florianduros florianduros requested a review from a team as a code owner February 19, 2025 09:36
@dbkr
Copy link
Member

dbkr commented Feb 19, 2025

Seems like this is just the non-chrome tests not working - merging manually.

@dbkr dbkr merged commit 28ed506 into develop Feb 19, 2025
47 of 55 checks passed
@dbkr dbkr deleted the florianduros/new-room-list/legacy-room-list-header branch February 19, 2025 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Task Tasks for the team like planning X-Run-All-Tests When applied to PRs, it'll run the full gamut of end-to-end tests on the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants