Closed slifty closed 6 months ago
Attention: Patch coverage is 83.33333%
with 9 lines
in your changes are missing coverage. Please review.
Project coverage is 88.30%. Comparing base (
b23b8a6
) to head (8cb2d14
).:exclamation: Current head 8cb2d14 differs from pull request most recent head d2c3c13. Consider uploading reports for the commit d2c3c13 to get more accurate results
Files | Patch % | Lines |
---|---|---|
src/handlers/usersHandlers.ts | 68.18% | 6 Missing and 1 partial :warning: |
...eryParameters/extractAuthenticationIdParameters.ts | 83.33% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I think all of the untested lines are of the category of "it's OK not to test them"
Verifying success on all of the same checks as @reefdog. Hitting approve!
This PR adds a new
/users
endpoint which will allow the front end to look up the id associated with their session.I decided to implement this with auth context awareness off the bat, which means administrative users can actually view ALL users in the system if they want.
Resolves #942