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

fix(ClipboardCopy): Add string[] type to children #11177

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Venefilyn
Copy link

@Venefilyn Venefilyn commented Nov 13, 2024

The ClipboardCopy component already handles string[] but didn't have it written in the props as valid.

What: Closes #11186

Additional issues:

@patternfly-build
Copy link
Contributor

patternfly-build commented Nov 13, 2024

@Venefilyn Venefilyn force-pushed the fix/string-array-clipboard-copy branch from ce0fdf3 to f1671dd Compare November 18, 2024 15:18
@Venefilyn
Copy link
Author

Actually it doesn't handle it correctly for some reason. @tlabaj do you happen to know what's happening?

image

The ClipboardCopy component already handles string[] but didn't have it
written in the props as valid.
@Venefilyn Venefilyn force-pushed the fix/string-array-clipboard-copy branch from f1671dd to 1325248 Compare November 18, 2024 15:19
@Venefilyn
Copy link
Author

Ah nvm found the issue, was inside componentDidUpdate() 🙏

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.

Bug - ClipboardCopy - TS type does not allow string[] children
2 participants