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.

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

Added Group related example #2054

Closed
wants to merge 3 commits into from

Conversation

pm-bhatt
Copy link

@pm-bhatt pm-bhatt commented Aug 5, 2024

I have added this comment because there was no guidance on the document where in it shows that how to assign specific groups in resource set

This will be helpful to implement PLoP when we want to grant access to help-desk admins to specific groups only rather then all groups.

@pm-bhatt pm-bhatt changed the title Added Group related example WIP : Added Group related example Aug 5, 2024
@pm-bhatt pm-bhatt changed the title WIP : Added Group related example Added Group related example Aug 5, 2024
@pm-bhatt
Copy link
Author

@duytiennguyen-okta : Request to kindly approve so that it can be merged please.

@duytiennguyen-okta
Copy link
Contributor

@pm-bhatt you need to sign your commit

@pm-bhatt
Copy link
Author

pm-bhatt commented Sep 8, 2024

Hi @duytiennguyen-okta

Apology for disturbing you again, I have now committed with Signed Signature with little modification. Can you please approve?

@pm-bhatt
Copy link
Author

Hi @duytiennguyen-okta

Apology for disturbing you again, I have now committed with Signed Signature with little modification. Can you please approve?

@pm-bhatt
Copy link
Author

pm-bhatt commented Nov 1, 2024

@duytiennguyen-okta @monde : Apology for disturbing you again, I have now committed with Signed Signature with little modification. Can you please approve?

@duytiennguyen-okta
Copy link
Contributor

@pm-bhatt I tried to approve but github still require the commit to be sign. I can just take your work and bring it in if that's easier for you

@pm-bhatt
Copy link
Author

pm-bhatt commented Nov 4, 2024

Yes sure, I don't have any problem

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