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

fix: add middleware for datadog #331

Merged
merged 1 commit into from
Oct 18, 2024
Merged

fix: add middleware for datadog #331

merged 1 commit into from
Oct 18, 2024

Conversation

alangsto
Copy link
Member

JIRA: COSMO-462

Description: In order to have custom spans in datadog, we need to add DeploymentMonitoringMiddleware to the exams middleware.

Merge checklist:

  • All reviewers approved
  • CI build is green
  • Changelog record added
  • Documentation updated (not only docstrings)
  • Commits are squashed

Post merge:

  • Delete working branch (if not needed anymore)

Copy link

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  edx_exams/apps/api/v1
  views.py
  edx_exams/apps/api/v1/tests
  test_views.py
  edx_exams/apps/core
  data.py
  models.py
  statuses.py
  views.py
  edx_exams/apps/core/management/commands
  bulk_add_course_staff.py
  edx_exams/apps/core/management/commands/test
  test_bulk_add_course_staff.py
  edx_exams/apps/core/tests
  test_api.py
  test_context_processors.py
  test_email.py
  test_handlers.py
  test_views.py
  edx_exams/apps/lti
  views.py
  edx_exams/apps/lti/signals
  handlers.py
  edx_exams/apps/lti/signals/tests
  test_handlers.py
  edx_exams/apps/lti/tests
  test_views.py
  edx_exams/apps/router/tests
  test_interop.py
  test_views.py
Project Total  

This report was generated by python-coverage-comment-action

@alangsto alangsto merged commit d5e994d into main Oct 18, 2024
5 checks passed
@alangsto alangsto deleted the alangsto/add_middleware branch October 18, 2024 14:22
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