-
Notifications
You must be signed in to change notification settings - Fork 31
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: Retired workflow on the list policy screens #1213
feat: Retired workflow on the list policy screens #1213
Conversation
b615886
to
7dcc0f9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved with a minor nit
@zwidekalanga In addition to the Figma mock discrepancies mentioned above regarding the "Learner Credit Management" budgets overview page route, it doesn't look like this PR is tackling anything from the Figma mocks on the budget detail page route.
|
45417b7
to
e07d2ab
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1213 +/- ##
==========================================
+ Coverage 85.33% 85.35% +0.01%
==========================================
Files 537 537
Lines 11695 11709 +14
Branches 2472 2480 +8
==========================================
+ Hits 9980 9994 +14
Misses 1663 1663
Partials 52 52 ☔ View full report in Codecov by Sentry. |
1984865
to
443148d
Compare
443148d
to
9b2ffd6
Compare
9b2ffd6
to
d67bee6
Compare
For all changes
Only if submitting a visual change