-
Notifications
You must be signed in to change notification settings - Fork 19
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: subsidy box points to learner credit #810
Conversation
7d6f634
to
8d64e38
Compare
01ece75
to
512b2db
Compare
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #810 +/- ##
==========================================
+ Coverage 84.87% 85.02% +0.15%
==========================================
Files 320 324 +4
Lines 6399 6671 +272
Branches 1552 1615 +63
==========================================
+ Hits 5431 5672 +241
- Misses 941 968 +27
- Partials 27 31 +4
☔ View full report in Codecov by Sentry. |
512b2db
to
3874186
Compare
@adamstankiewicz Hi, I have incorporated all your feedback. Can you please take another look? |
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.
LGTM with a suggestion and a clarifying question!
3874186
to
0c9031b
Compare
added tests added more tests minor fixes incorporated feedback incorporated feedback round 2 minor change in the test
0c9031b
to
73ad5ac
Compare
This reverts commit 3637bed.
For all changes
Only if submitting a visual change