-
Notifications
You must be signed in to change notification settings - Fork 246
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
feat: Collecting community messages count metrics #3802
Conversation
Jenkins BuildsClick to see older builds (40)
|
67a635b
to
d3ab3e1
Compare
7025645
to
dd39ac8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 👍
dd39ac8
to
5b8a832
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice job. This is very clean
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very well done @MishkaRogachev! 🚀
c16a95e
to
b2ace59
Compare
For status-im/status-desktop#11152
Implement both strategies for fetching community messaging metrics:
CommunityMetricsRequestMessagesTimestamps
to collect messages timestamps by intervals for all community chatsCommunityMetricsRequestMessagesCount
to collect messages count by intervals for all community chats