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 validations for the cluster parameter in API #1100

Closed

Conversation

rdrck47
Copy link
Contributor

@rdrck47 rdrck47 commented May 25, 2022

Hello! Here is a PR that targets issue #1094.
I think the proposed solution that I developed could be better, I am trying to improve the solution implementation without increasing code complexity, in order to avoid introducing new bugs.

  • Adds validations in the API for checking that the provided cluster-name parameter is configured

Let me know if you got any questions or suggestions to improve my solution.

@tchiotludo
Copy link
Owner

superseded by #1104

@tchiotludo tchiotludo closed this May 26, 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