Skip to content
This repository was archived by the owner on Sep 11, 2024. It is now read-only.

Conversation

@andybalaam
Copy link
Member

@andybalaam andybalaam commented Sep 7, 2023

As agreed in our weekly meeting.


This change is marked as an internal change (Task), so will not be included in the changelog.

@andybalaam andybalaam added the T-Task Refactoring, enabling or disabling functionality, other engineering tasks label Sep 7, 2023
@andybalaam andybalaam requested a review from a team as a code owner September 7, 2023 16:56
@andybalaam andybalaam enabled auto-merge September 8, 2023 02:25
@andybalaam andybalaam added this pull request to the merge queue Sep 10, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 10, 2023
@andybalaam
Copy link
Member Author

This uncovered flakiness in the spotlight tests, which should be already fixed by #11590

It also uncovered flakiness in "An edited message becomes read if it happens while I am looking" in the high-level read receipt tests, so that needs looking into before we merge this.

@andybalaam
Copy link
Member Author

First time we ran the tests, 2 cypress tests failed.

Second time, 1 cypress test failed (audio-player).

Running a third time, but it looks like we are not ready for this yet.

@andybalaam
Copy link
Member Author

The third time, the spotlight tests failed.

So I think we are not ready for this yet.

@andybalaam andybalaam closed this Nov 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

T-Task Refactoring, enabling or disabling functionality, other engineering tasks

3 participants