-
Notifications
You must be signed in to change notification settings - Fork 2
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: add users endpoint #163
Conversation
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.
tests gaan wel fout
68c35dd
to
cdc59dc
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #163 +/- ##
==========================================
- Coverage 92.93% 91.48% -1.45%
==========================================
Files 96 100 +4
Lines 2249 2326 +77
Branches 125 128 +3
==========================================
+ Hits 2090 2128 +38
- Misses 34 70 +36
- Partials 125 128 +3 ☔ View full report in Codecov by Sentry. |
cdc59dc
to
f55ff26
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.
coverage moet nog
public readonly path = 'users'; | ||
public readonly property = 'users'; | ||
} | ||
|
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.
jouw editor lijkt iets geks te doen met newlines aan het eind
f55ff26
to
741463a
Compare
741463a
to
5933f39
Compare
5933f39
to
e6494ae
Compare
a4c14e3
to
73f396a
Compare
73f396a
to
00f0b3d
Compare
6ae8680
to
76059c9
Compare
Pull request was closed
Adds GET endpoints for 1) all users with pagination 2) single/multiple user endpoint