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.

Sign up for GitHub

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

Jump to bottom

Project Editor: Whitelisted Cluster Resources doesn't strip whitespace #1693

Closed
stgarf opened this issue Jun 5, 2019 · 1 comment · Fixed by #1722
Closed

Project Editor: Whitelisted Cluster Resources doesn't strip whitespace #1693

stgarf opened this issue Jun 5, 2019 · 1 comment · Fixed by #1722
Labels
enhancement New feature or request good first issue Good for newcomers
Milestone
v1.1

Comments

@stgarf
Copy link
Contributor

stgarf commented Jun 5, 2019

In the project UI for adding whitelisted cluster resources, upon saving, whitespace is not trimmed from the fields

untrimmed_whitespace
@alexec alexec added enhancement New feature or request good first issue Good for newcomers labels Jun 5, 2019
@stgarf
Copy link
Contributor Author

stgarf commented Jun 6, 2019

Just clarifying that it's a UI bug (not an enhancement request) where whitespace is not stripped pre- or post- string.

@AdityaGupta1 AdityaGupta1 mentioned this issue Jun 10, 2019
Merged
alexec pushed a commit that referenced this issue Jun 10, 2019
@AdityaGupta1 @alexec
Issue #1693 - Project Editor: Whitelisted Cluster Resources doesn't s…
4dc102a 
…trip whitespace (#1722)
@alexec alexec added this to the v1.1 milestone Jun 14, 2019
@crenshaw-dev crenshaw-dev mentioned this issue Jul 25, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
v1.1
Development

Successfully merging a pull request may close this issue.

Issue #1693 - Project Editor: Whitelisted Cluster Resources doesn't s…
2 participants
@alexec @stgarf