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

feat(exoflex): add testID to some components #405

Merged
merged 2 commits into from
Mar 19, 2020

Conversation

oshimayoan
Copy link
Contributor

Resolves #384.

Honestly, I'm still not sure if it's the best solution for RichRadioButton component.

@darcien
Copy link
Contributor

darcien commented Mar 18, 2020

Honestly, I'm still not sure if it's the best solution for RichRadioButton component.

Looks fine to me, what's your concern?

@oshimayoan
Copy link
Contributor Author

Well, because I put the testID inside the data object.

@darcien
Copy link
Contributor

darcien commented Mar 19, 2020

Well, because I put the testID inside the data object.

Ahh right, but in #409 we separated the Item and the Group right, so we don't have to pass the testID together with the data.

@oshimayoan
Copy link
Contributor Author

Yeah, that's true tho. I guess we could land this now.

@oshimayoan oshimayoan merged commit ff37001 into kodefox:master Mar 19, 2020
@oshimayoan oshimayoan added this to the exoflex v3.3.0 milestone Mar 30, 2020
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.

exoflex: testID not found from props
3 participants