Hyperion
Hyperion copied to clipboard
Disable account
Description
I created code to be able to delete accounts that do not have a Central address. To do that, I :
- Added a Boolean enabled attribute to the CoreUser
- Added a CRUD to deactivate a CoreUser
- Have prevented disabled CoreUsers from connecting
I still have to exclude deactivated CoreUsers from the search
Codecov Report
Attention: Patch coverage is 79.41176% with 7 lines in your changes are missing coverage. Please review.
Project coverage is 78.32%. Comparing base (
3cb0984) to head (37fb85b). Report is 12 commits behind head on main.
:exclamation: Current head 37fb85b differs from pull request most recent head ceb1f52. Consider uploading reports for the commit ceb1f52 to get more accurate results
| Files | Patch % | Lines |
|---|---|---|
| app/cruds/cruds_external_account.py | 75.00% | 3 Missing :warning: |
| app/endpoints/auth.py | 0.00% | 2 Missing :warning: |
| app/endpoints/users.py | 0.00% | 2 Missing :warning: |
Additional details and impacted files
@@ Coverage Diff @@
## main #344 +/- ##
==========================================
+ Coverage 78.30% 78.32% +0.01%
==========================================
Files 78 80 +2
Lines 5435 5467 +32
==========================================
+ Hits 4256 4282 +26
- Misses 1179 1185 +6
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.