-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add manage content permissions recipe #65
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.
just a few minor nits from me :)
Co-authored-by: annikakrilov <[email protected]>
@kmasiello I think this is good to go now, not sure if CI should pass, merge when its all good on your end? |
Yeah IDK what's wrong with the GH pages build but I don't see this locally. We probably need to disable it until we sort it out. |
It's a shame this has to be 4 distinct recipes. See rstudio/connectapi#267 and posit-dev/posit-sdk-py#194 requesting that adding permissions can flexibly take either a user or group guid. Something that could be improved upon in a later iteration. |
Closes #49