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

Workaround for Create React App (CRA v1) #24

Closed

Conversation

JVannucc
Copy link

See issue #23, which references, szimek/signature_pad#257.

There is a bug in Create React App, that will cause a crash because of the signature_pad import.

The workaround suggested is to import the js file directly.

@agilgur5
Copy link
Owner

agilgur5 commented Oct 21, 2018

@JVannucc thanks for your contribution!

Per the comment I just left in #23 , I'm not sure that it makes sense for this library to implement a workaround for a bug that isn't here or in upstream signature_pad, but rather only in certain developers' build tools, namely CRA v1. If aliasing works, then I think that should be the go-to solution for users who are facing bugs with CRA v1 -- fix build tools with build tools instead of patching all sorts of libraries that could be affected by the build tools' bugs.

@JVannucc
Copy link
Author

No worries.

Cheers!

@JVannucc JVannucc closed this Oct 21, 2018
@agilgur5 agilgur5 changed the title Import Signature pad from it's source file. Workaround for Create React App (CRA v1) Oct 21, 2018
@agilgur5 agilgur5 added solution: wontfix This will not be changed at this time scope: upstream Issue in upstream dependency solution: out-of-scope This is out of scope for this project and removed solution: wontfix This will not be changed at this time labels Mar 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: upstream Issue in upstream dependency solution: out-of-scope This is out of scope for this project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants