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

Update validCustomRolePermissions to include okta.workflows.invoke #1160

Merged
merged 3 commits into from
Jun 9, 2022

Conversation

tim-fitzgerald
Copy link
Contributor

@tim-fitzgerald tim-fitzgerald commented Jun 9, 2022

This PR adds the permission set okta.workflows.invoke as an allowable Custom Role Permission.

This is based on the new feature from Okta to allow delegated workflows via the standard admin panel.

Release Notes
Screen Shot 2022-06-09 at 9 49 53 AM

This is an early access feature that requires the Admin to enable 3 feature flags in their admin panel, I was not sure how best to document this requirement in the documentation.

Required Features:
Enable Delegated Flows card type in Workflows
Okta Resource Name (ORN) in API for Administrator Roles
Delegate workflows with custom admin roles

@monde monde self-requested a review June 9, 2022 18:11
@monde
Copy link
Collaborator

monde commented Jun 9, 2022

Thanks @tim-fitzgerald can you do me a favor and add all of the new permission properties to this PR? I'm trying to do a v3.29.0 PR today and would like to get this in the release. Thanks!

https://developer.okta.com/docs/reference/api/roles/#permission-properties

That would also be:

  • "okta.authzServers.manage" : Allows the admin to manage authorization servers
  • "okta.authzServers.read" : Allows the admin to read authorization servers
  • "okta.customizations.manage" : Allows the admin to manage customizations
  • "okta.customizations.read" : Allows the admin to read customizations

@tim-fitzgerald
Copy link
Contributor Author

@monde for sure - will be free in about 2 hours and can add them then.

@tim-fitzgerald
Copy link
Contributor Author

@monde added those additional permissions sets.

Copy link
Collaborator

@monde monde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @tim-fitzgerald

@monde monde merged commit 82febe3 into okta:master Jun 9, 2022
@monde
Copy link
Collaborator

monde commented Jun 10, 2022

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