Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add GetUsers endpoint #75

Merged
merged 51 commits into from
Dec 6, 2023
Merged

Add GetUsers endpoint #75

merged 51 commits into from
Dec 6, 2023

Conversation

MadsSR
Copy link
Contributor

@MadsSR MadsSR commented Dec 5, 2023 •
edited
Loading

  • Add GetUsers endpoint
  • Delete session when refresh token is expired
  • Minor changes to TokenError
  • Organize functions in ecdar_api.rs

ViktorPlatz and others added 28 commits December 4, 2023 12:16
@ViktorPlatz
protobuf
c8b14c1
@sabotack
Add role check to create_access and fix tests
d1d7a05
@sabotack
Move comment
08719c3
@ViktorPlatz
add list access info endpoint
df0b9c8
@ViktorPlatz
add FromQueryResult so database can convert
78ddaec
@ViktorPlatz
list access info endpoints tests
cde61b0
@ViktorPlatz
get_access_by_model_id crud
10bcc62
@ViktorPlatz
get_access_by_model_id crud tests
2832c6c
@ViktorPlatz
protobufffffffffffffffffff
77f9db1
@ViktorPlatz
clippy
3bbfba3
@sabotack
Implement role checking in create_access
e77f78d
@ViktorPlatz
protobuf stuff
1a577f1
@sabotack
Implement role checking in update_access
f6a4b7d
@sabotack
Add editor_role_check_helper method
f87e1c0
@sabotack
Implement role checking in delete_access
b79a3d0
@ViktorPlatz
merge with alis branch
32baa72
@ViktorPlatz
clippy fmt
573445e
@MadsSR
Add thiserror crate
4910565
@MadsSR
Add get_by_ids mock
e85d910
@MadsSR
Fix tests
fead523
@MadsSR
Add get users endpoint
c2d7276
@MadsSR
Add get_by_ids on user context
cb125ba
@MadsSR
Fix Token
7923c4e
@MadsSR
Fix path to Model
0afc0c2
@MadsSR
Verify hashed password and refactor
01ef8b5
@MadsSR
Add get auth token tests
c9547f5
@MadsSR
Add GetUsers rpc to protobuff
cd29928
@MadsSR
WIP: Fix session_logic test
8aa7e93
@MadsSR MadsSR changed the title Get users Add GetUsers endpoint Dec 5, 2023
@MadsSR MadsSR marked this pull request as ready for review December 6, 2023 11:02
@MadsSR MadsSR requested a review from williamwoldum December 6, 2023 11:02
@MadsSR MadsSR added the enhancement New feature or request label Dec 6, 2023
@MadsSR
Copy link
Contributor Author

MadsSR commented Dec 6, 2023

LGTM

@williamwoldum williamwoldum merged commit a08f88e into main Dec 6, 2023
6 checks passed
@williamwoldum williamwoldum deleted the GetUsers branch December 6, 2023 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@williamwoldum williamwoldum williamwoldum approved these changes

Assignees
No one assigned
Labels
A$AP enhancement New feature or request ROCKY
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@MadsSR @williamwoldum @ViktorPlatz @sabotack