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

Allow passing arbitrary props to ReText component #501

Merged
merged 2 commits into from Sep 12, 2022

Conversation

Jyrno42
Copy link
Contributor

@Jyrno42 Jyrno42 commented Sep 7, 2022

fix #500

@wcandillon
Copy link
Owner

Looks great, could the same be achieve with TextProps extends TextInputProps?

@Jyrno42
Copy link
Contributor Author

Jyrno42 commented Sep 7, 2022

I used a separate prop since I wanted to avoid people setting "value" prop via this. But now that I read it it's still possible to do that. I guess changing it to be TextProps extends Omit<TextInputProps, 'value'> would be the best solution.

If that works I'll make the change and then will verify it in my app too.

@wcandillon
Copy link
Owner

yes that sounds awesome

@Jyrno42 Jyrno42 changed the title Add textInputProps to ReText component to allow passing arbitrary props to it Allow passing arbitrary props to ReText component Sep 8, 2022
@Jyrno42
Copy link
Contributor Author

Jyrno42 commented Sep 8, 2022

It's updated now and worked in my app too.

@wcandillon wcandillon merged commit d338689 into wcandillon:master Sep 12, 2022
@wcandillon
Copy link
Owner

so good thks!

@wcandillon
Copy link
Owner

🎉 This PR is included in version 18.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow passing arbitrary props like testID to the ReText TextInput component
2 participants