Skip to content

Refactor: Class-based views - Enrollment Index - #3073

Merged
lalver1 merged 1 commit into
mainfrom
refactor/cbv-enrollment-index
Aug 11, 2025
Merged

Refactor: Class-based views - Enrollment Index#3073
lalver1 merged 1 commit into
mainfrom
refactor/cbv-enrollment-index

Conversation

@lalver1

@lalver1 lalver1 commented Aug 8, 2025

Copy link
Copy Markdown
Member

Closes #3015

This PR converts the enrollment index view function into a RedirectView class-based view.

@lalver1 lalver1 self-assigned this Aug 8, 2025
@github-actions github-actions Bot added tests Related to automated testing (unit, UI, integration, etc.) back-end Django views, sessions, middleware, models, migrations etc. and removed tests Related to automated testing (unit, UI, integration, etc.) labels Aug 8, 2025
@github-actions

github-actions Bot commented Aug 8, 2025

Copy link
Copy Markdown
Contributor

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  benefits/enrollment
  urls.py
  views.py
Project Total  

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

@lalver1
lalver1 marked this pull request as ready for review August 8, 2025 15:03
@lalver1
lalver1 requested a review from a team as a code owner August 8, 2025 15:03

@thekaveman thekaveman left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, was able to run through enrollment locally (Switchio)

@lalver1
lalver1 merged commit 40f8956 into main Aug 11, 2025
17 checks passed
@lalver1
lalver1 deleted the refactor/cbv-enrollment-index branch August 11, 2025 14:36
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Class-based views: Enrollment index

2 participants