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

Remove unnecessary dependency to Identity Util to read action execution configs #5864

Merged
merged 4 commits into from
Aug 15, 2024

Conversation

malithie
Copy link
Member

Proposed changes in this pull request

Resolves wso2/product-is#20915

@ashanthamara
Copy link
Contributor

Shall we set this config to true since we are enabling the feature by default. As the Actions UI is also there in the console, user might get confused if he missed to set this config to true.

And do we need to mention about this config in Identity Server docs ?

@malithie
Copy link
Member Author

Shall we set this config to true since we are enabling the feature by default. As the Actions UI is also there in the console, user might get confused if he missed to set this config to true.

And do we need to mention about this config in Identity Server docs ?
This is a feature flag. Therefore, it makes sense to add it if we are having a config catalog. But looks like we don't have such. In that case IMO we don't need to mention this if the feature related doc

@malithie malithie merged commit e144108 into wso2:master Aug 15, 2024
2 checks passed
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.

Remove unnecessary dependency to Identity Util to read action execution configs
2 participants