Skip to content

Add function to retrieve current user's workspace permission#53

Merged
naglepuff merged 5 commits intogirder4from
permissions-current-user
Aug 20, 2021
Merged

Add function to retrieve current user's workspace permission#53
naglepuff merged 5 commits intogirder4from
permissions-current-user

Conversation

@naglepuff
Copy link
Contributor

This PR is dependent on this multinet-api PR. See that PR for specific details regarding the new API endpoint.

This PR works in conjunction with the above API PR to allow for retrieval of the permission of the currently logged in user for a given workspace.

Changes:

  1. New SingleUserWorkspacePermissionSpec to describe the server's response for a GET /api/workspaces/{workspace}/permissions/me request
  2. New functions to allow our custom axios instance to make relevant requests

@naglepuff naglepuff requested a review from jjnesbitt August 20, 2021 15:13
Since the permission property can be null, its type should be 'string | null'.

Co-authored-by: Jacob Nesbitt <jjnesbitt2@gmail.com>
@naglepuff naglepuff requested a review from jjnesbitt August 20, 2021 15:24
@naglepuff naglepuff merged commit 5e625c9 into girder4 Aug 20, 2021
@naglepuff naglepuff deleted the permissions-current-user branch August 20, 2021 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants