-
Notifications
You must be signed in to change notification settings - Fork 78
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(Community): Community messaging statistics chart #11696
feat(Community): Community messaging statistics chart #11696
Conversation
Jenkins BuildsClick to see older builds (61)
|
441e485
to
25def43
Compare
e0e4d1f
to
9539765
Compare
Close 11152 - Use se `collectCommunityMessageMetrics` for messaging statistics chart in community overview
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.
I'm requesting changes just for the wrong module.
9539765
to
afa8815
Compare
Added an issue for further work: #11758 |
4fc0f47
to
f2d07b6
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.
Looks good. Just one comment if you could
f2d07b6
to
dfb5377
Compare
dfb5377
to
20313df
Compare
Co-authored-by: Jonathan Rainville <rainville.jonathan@gmail.com>
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! Tested and works nicely! 👍
Close #11152
Waits status-im/status-go#3802
What does the PR do
collectCommunityMessageMetrics
for messaging statistics chart in community overviewAffected areas
Communities
Screenshot of functionality (including design for comparison)
Screen.Recording.2023-08-02.at.18.14.36.mov