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

Add statistics endpoint for actstream actions #2390

Merged
merged 2 commits into from
Apr 20, 2023

Conversation

DavisRayM
Copy link
Contributor

@DavisRayM DavisRayM commented Mar 6, 2023

Changes / Features implemented

  • Add MessagingStatsViewset class
  • Add tests for the messaging stats viewset

Steps taken to verify this change does what is intended

  • QA

Side effects of implementing this change

N/A

Before submitting this PR for review, please make sure you have:

  • Included tests
  • Updated documentation

@KipSigei
Copy link
Contributor

KipSigei commented Mar 6, 2023

@DavisRayM could we add documenation for the new endpoint?

@DavisRayM DavisRayM force-pushed the messaging-timeseries-data branch 3 times, most recently from a559e16 to d2cdc1f Compare March 13, 2023 09:42
@DavisRayM DavisRayM requested a review from KipSigei March 13, 2023 09:42
@DavisRayM DavisRayM force-pushed the messaging-timeseries-data branch 2 times, most recently from ff8d4b4 to 1a5f7ec Compare March 13, 2023 09:55
@DavisRayM DavisRayM force-pushed the messaging-timeseries-data branch 2 times, most recently from 220b667 to 67a1151 Compare April 18, 2023 11:38
KipSigei
KipSigei previously approved these changes Apr 18, 2023
@DavisRayM DavisRayM force-pushed the messaging-timeseries-data branch 5 times, most recently from 7094d72 to 02d2a17 Compare April 18, 2023 13:13
KipSigei
KipSigei previously approved these changes Apr 18, 2023
@DavisRayM DavisRayM merged commit 9ac7b2c into main Apr 20, 2023
@DavisRayM DavisRayM deleted the messaging-timeseries-data branch April 20, 2023 07:15
@DavisRayM DavisRayM mentioned this pull request May 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants