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

Report loading events for progress table to new relic #45095

Merged
merged 1 commit into from Mar 2, 2022

Conversation

maureensturgeon
Copy link
Contributor

@maureensturgeon maureensturgeon commented Mar 1, 2022

Adds some logging of events to New Relic so that we can get a better sense of how our teacher dashboard progress table is performing.

The logs in the console represent events that will be reported to New Relic:
https://user-images.githubusercontent.com/24235215/156222571-9bb98545-636a-4c67-b03e-407898e684f7.mov

Links

Testing story

I plan on taking a look in New Relic once these changes are released to figure out if the reporting that I've added is enough or if we need more information. It wouldn't surprise me if we need to do some follow-up work to add more reporting or more specific data.

PR Checklist:

  • Tests provide adequate coverage
  • Privacy and Security impacts have been assessed
  • Code is well-commented
  • New features are translatable or updates will not break translations
  • Relevant documentation has been added or updated
  • User impact is well-understood and desirable
  • Pull Request is labeled appropriately
  • Follow-up work items (including potential tech debt) are tracked and linked

@maureensturgeon maureensturgeon marked this pull request as ready for review March 1, 2022 18:00
@maureensturgeon maureensturgeon requested a review from a team March 1, 2022 18:00
@maureensturgeon maureensturgeon merged commit 8255878 into staging Mar 2, 2022
@maureensturgeon maureensturgeon deleted the maureen/LP-2203-nr-report-progress-table branch March 2, 2022 00:38
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.

None yet

2 participants