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: handle enrollment errors with retry view #1936

Closed
wants to merge 2 commits into from

Conversation

thekaveman
Copy link
Member

Closes #1903

Copy link

github-actions bot commented Mar 7, 2024

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  benefits/core
  models.py
  benefits/enrollment
  analytics.py
  middleware.py
  views.py
Project Total  

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

@github-actions github-actions bot added back-end Django views, sessions, middleware, models, migrations etc. tests Related to automated testing (unit, UI, integration, etc.) labels Mar 7, 2024
move retry template name to middleware module

middleware will be used as a decorator on relevant enrollment views
Base automatically changed from refactor/enrollment-backoffice to dev March 8, 2024 20:37
@thekaveman thekaveman linked an issue Mar 11, 2024 that may be closed by this pull request
1 task
@thekaveman thekaveman self-assigned this Mar 11, 2024
@thekaveman thekaveman added this to the Back Office API Enrollment milestone Mar 11, 2024
@thekaveman
Copy link
Member Author

I'm closing this PR based on our discussion around needing a more formal design/dev process for Error pages. New tickets will be coming likely for the April sprint timeline.

I'll keep the branch around in case we can reuse any of this work.

@thekaveman thekaveman closed this Mar 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
back-end Django views, sessions, middleware, models, migrations etc. tests Related to automated testing (unit, UI, integration, etc.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enrollment errors should redirect user to the Payment Error page / retry view
1 participant