-
Notifications
You must be signed in to change notification settings - Fork 28
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
don't save student status to owner if it is false #180
Conversation
Codecov Report
@@ Coverage Diff @@
## main #180 +/- ##
=====================================
Coverage 95.51 95.51
=====================================
Files 714 714
Lines 15584 15589 +5
=====================================
+ Hits 14884 14889 +5
Misses 700 700
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #180 +/- ##
=======================================
Coverage 95.61% 95.61%
=======================================
Files 600 600
Lines 15183 15183
=======================================
Hits 14517 14517
Misses 666 666
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Codecov Report
@@ Coverage Diff @@
## main #180 +/- ##
=======================================
Coverage 95.61% 95.61%
=======================================
Files 600 600
Lines 15183 15183
=======================================
Hits 14517 14517
Misses 666 666
Flags with carried forward coverage won't be shown. Click here to find out more.
|
9ec491e
to
c25b6ae
Compare
c25b6ae
to
820e2ab
Compare
Signed-off-by: joseph-sentry <joseph.sawaya@sentry.io>
Signed-off-by: joseph-sentry <joseph.sawaya@sentry.io>
820e2ab
to
b30afc8
Compare
* Don't update owner student if student_disabled is True * fix tests for student login Signed-off-by: joseph-sentry <joseph.sawaya@sentry.io>
Purpose/Motivation
We don't want to save the student status of an owner if it's false
Links to relevant tickets
Fixes: codecov/engineering-team#644
What does this PR do?