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

Remove old React version and import React, ReactDOM and createReactCl… #324

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

elchininet
Copy link

@elchininet elchininet commented Sep 10, 2019

Remove old React version and import React, ReactDOM and createReactClass directly from the entry-points

In this commit (af4d9a8) React and ReactDOM where added as dependencies of the project
But it seems that they are not used because no entry-point import them. The React version that is used is the version 0.12.2 that is coming from 2014 and it is loaded as a separate asset.
The purpose of this commit is to import React, ReactDOM, and createReactClass from the entry-points and get rid of the old react version.
In this way, the project can be escalated using the new features of React

…ass directly from the entry-points

In this commit (af4d9a8) React and ReactDOM where added as dependencies of the project
But it seems that they are not used because no entry-point import them. The React version that is used is the version 0.12.2 that is coming from 2014 and it is loaded as a separate script directly from the HTML templates.
The purpose of this commit is to import React, ReactDOM, and createReactClass from the entry-points and get rid of the old react version.
In this way, the project can be escalated using the new features of React
hvariant pushed a commit to hvariant/hound that referenced this pull request Oct 5, 2019
When hound search returns a large number of results it can be annoying to
scroll through the everything to find what you need. This adds a button to
collapse file body in search results.

Note that this change depends on hound-search#324 which adds new version of React which
has support for React.createRef.
Base automatically changed from master to main February 24, 2021 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants