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

Adds a Troubleshooting section to README.md #124

Merged
merged 2 commits into from
Apr 7, 2016
Merged

Adds a Troubleshooting section to README.md #124

merged 2 commits into from
Apr 7, 2016

Conversation

cguedes
Copy link
Contributor

@cguedes cguedes commented Apr 6, 2016

Troubleshooting guidelines to handle the build error: Cannot find module 'buffertools' from .../node_modules/react-jsonschema-form/node_modules/deeper

Troubleshooting guidelines to handle the build error: Cannot find module 'buffertools' from .../node_modules/react-jsonschema-form/node_modules/deeper
@cguedes cguedes mentioned this pull request Apr 6, 2016
@n1k0
Copy link
Collaborator

n1k0 commented Apr 6, 2016

Thanks. I'm gonna hold on merging this as I've just submitted othiym23/node-deeper#4

If my PR is ever merged, we won't need specific config like this anymore, which would be nice.

@cguedes
Copy link
Contributor Author

cguedes commented Apr 6, 2016

Ok, unless someone is using this specific version of react-jsonschema-form.

@n1k0
Copy link
Collaborator

n1k0 commented Apr 6, 2016

That's very true. Let's land this for further reference. But before some small nits to address :)


If you use **webpack** you can do the following to ignore this import:

~~~javascript
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use the same md syntax for code blocks, as used everywhere else in the file for consistency.

@n1k0 n1k0 merged commit 0a20c44 into rjsf-team:master Apr 7, 2016
@n1k0
Copy link
Collaborator

n1k0 commented Apr 7, 2016

Thank you!

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.

2 participants