-
Notifications
You must be signed in to change notification settings - Fork 309
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
fix: Remove 3PI config url validation #1220
Conversation
nit: Using the |
Please update the README to mention that we do not validate the URLs in the config for Workload/Workforce |
I considered that. But I found those validations are not added the same time. And very hard to find all these separate changes. |
Hi @lsirac , and for the |
Let's keep it for now, and circle back on this |
Good. Maybe need backlog a "loose the pattern check for is_workforce_pool" |
@arithmetic1728 may I get an approval? |
Remove the url validation for TPC support.
Failed configuration will be on user's own risk in future.