-
Notifications
You must be signed in to change notification settings - Fork 669
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
NOISSUE - Fix Share Thing To Add External To Request #1886
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1886 +/- ##
==========================================
- Coverage 67.12% 67.11% -0.02%
==========================================
Files 118 118
Lines 9011 9020 +9
==========================================
+ Hits 6049 6054 +5
- Misses 2333 2337 +4
Partials 629 629
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Signed-off-by: rodneyosodo <blackd0t@protonmail.com>
93a3962
to
6bcdd4f
Compare
pkg/sdk/go/policies.go
Outdated
@@ -21,6 +21,7 @@ type Policy struct { | |||
Subject string `json:"subject"` | |||
Object string `json:"object"` | |||
Actions []string `json:"actions"` | |||
External bool `json:"external,omitempty"` |
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.
There probably should be a comment explaining what this External
means, as here it looks like it means something like "external policy"
Signed-off-by: rodneyosodo <blackd0t@protonmail.com>
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.
LGTM
Signed-off-by: rodneyosodo blackd0t@protonmail.com
What does this do?
Enables sharing things on SDK level
Which issue(s) does this PR fix/relate to?
No issue
List any changes that modify/break current functionality
None
Have you included tests for your changes?
No
Did you document any new/modified functionality?
No
Notes
Add
external
field to policies. This will enables us to share things between users