- Notifications
You must be signed in to change notification settings - Fork928
feat: Add endpoint to get all workspaces a user can access#1354
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Conversation
codecovbot commentedMay 10, 2022 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Codecov Report
@@ Coverage Diff @@## main #1354 +/- ##==========================================- Coverage 67.03% 66.73% -0.31%========================================== Files 284 284 Lines 18529 18614 +85 Branches 235 235 ==========================================+ Hits 12421 12422 +1- Misses 4844 4925 +81- Partials 1264 1267 +3
Continue to review full report at Codecov.
|
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
| @@ -56,6 +56,7 @@ require ( | |||
| github.com/fatedier/frp v0.42.0 | |||
| github.com/fatedier/golib v0.1.1-0.20220321042308-c306138b83ac | |||
| github.com/fatih/color v1.13.0 | |||
| github.com/fatih/structs v1.1.0 | |||
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.
oopsie, i mightve forgotten to go mod tidy. we should have a ci step for this, i'll do it in a bit
| @@ -1,279 +0,0 @@ | |||
| package main | |||
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.
I imagine this is unrelated
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.
It is, but figured I'd clean it up in here.
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.
yup, makes sense
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.
Correct. Figured it was easier to remove this here. It hasn't been used in forever, and is kinda sketchy code.
This iterates through user organizations to get permittedworkspaces. This will allow admins to manage user workspaces!
This iterates through user organizations to get permittedworkspaces. This will allow admins to manage user workspaces!
This iterates through user organizations to get permitted
workspaces. This will allow admins to manage user workspaces!