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

Add analytics for mark all threads unread #12384

Merged
merged 2 commits into from
Apr 4, 2024
Merged

Add analytics for mark all threads unread #12384

merged 2 commits into from
Apr 4, 2024

Conversation

dbkr
Copy link
Member

@dbkr dbkr commented Mar 28, 2024

Requires matrix-org/matrix-analytics-events#100

Checklist

  • Tests written for new code (and old code if feasible).
  • New or updated public/exported symbols have accurate TSDoc documentation.
  • Linter and other CI checks pass.
  • Sign-off given on the changes (see CONTRIBUTING.md).

@dbkr dbkr added the T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements label Mar 28, 2024
@dbkr dbkr marked this pull request as ready for review April 2, 2024 10:44
@dbkr dbkr requested review from a team as code owners April 2, 2024 10:44
@dbkr dbkr requested review from t3chguy and richvdh April 2, 2024 10:44
richvdh
richvdh approved these changes Apr 3, 2024
View reviewed changes
Copy link
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

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

Seems plausible

@dbkr dbkr added this pull request to the merge queue Apr 4, 2024
Merged via the queue into develop with commit 307d737 Apr 4, 2024
22 checks passed
@dbkr dbkr deleted the dbkr/mark_all_threads_read_analytics branch April 4, 2024 12:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@florianduros florianduros florianduros approved these changes

@richvdh richvdh richvdh approved these changes

@t3chguy t3chguy Awaiting requested review from t3chguy t3chguy is a code owner automatically assigned from matrix-org/element-web-team

Assignees
No one assigned
Labels
T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dbkr @richvdh @florianduros