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

Added firehose logging so we can track how students are pairing. #35933

Merged
merged 2 commits into from
Jul 22, 2020

Conversation

jmkulwik
Copy link
Contributor

Allows us to track how many other users a user is pairing with. Additionally, allows us to track which section, level, and script a user initiated pairing from.

Note: the firehose client wrapper has an existing limitation where it is unable to record the user_id of a user when pairing is initiated from the user's dashboard.

Links

Testing story

Reviewer Checklist:

  • Tests provide adequate coverage
  • 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

@jmkulwik jmkulwik merged commit d206164 into staging Jul 22, 2020
@jmkulwik jmkulwik deleted the track-pairing branch July 22, 2020 16:41
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