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

feat: add middleware routing for onboarding endpoint #355

Merged
merged 1 commit into from
Jan 29, 2025

Conversation

alangsto
Copy link
Member

JIRA: https://2u-internal.atlassian.net/browse/COSMO-623

Description: This PR adds support to route onboarding requests to the edx-proctoring backend via the edx-exams middleware. Because edx-exams does not support onboarding exams at this point, any requests routed to the edx-exams backend will return a 404 response.

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/router
  interop.py
  middleware.py
  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 force-pushed the alangsto/add_onboarding_endpoint_to_middleware branch from c44eb95 to 1cd6e8f Compare January 29, 2025 16:25
Copy link
Member

@schenedx schenedx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@alangsto alangsto merged commit 1109ef3 into main Jan 29, 2025
4 checks passed
@alangsto alangsto deleted the alangsto/add_onboarding_endpoint_to_middleware branch January 29, 2025 16:33
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