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

Add Live Demo #92

Closed
wants to merge 1 commit into from
Closed

Add Live Demo #92

wants to merge 1 commit into from

Conversation

KimGenius
Copy link

Hi!

Thank you so much to made good library

I think so this library need Live Demo code link in README !
So, i made Live Demo with Code sandbox

If you think like me, please merge this pull request!

Thank you!

Add Live demo link
@demerino
Copy link
Member

demerino commented May 8, 2019

@KimGenius Thanks for the thoughtful demo and PR! This seems like a great idea, we'll review the demo and see what is the best path forward for linking to live demos.

@crookedneighbor
Copy link
Contributor

I like the idea of adding a demo, but I think it should:

A) be presented in this repo through Github Pages so it can be reached at https://braintree.github.io/credit-card-type

B) be framework agnostic, not specifically React

@crookedneighbor
Copy link
Contributor

Going to close this for now, but if you'd like to submit a new PR that meets the criteria I laid out, go for it.

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.

None yet

3 participants