Skip to content
This repository has been archived by the owner on Jun 17, 2024. It is now read-only.

Upgrade deps; include React Hooks #55

Open
wagnerjt opened this issue Apr 9, 2020 · 2 comments
Open

Upgrade deps; include React Hooks #55

wagnerjt opened this issue Apr 9, 2020 · 2 comments

Comments

@wagnerjt
Copy link
Contributor

wagnerjt commented Apr 9, 2020

Is your feature request related to a problem? Please describe.
N/A

Describe the solution you'd like
Upgrade all dependencies (focusing more on the react-scripts) to get react hooks in. Many things have changed in the React world with the lifecycle methods and it would be best to use them for the neo4j desktop integration to subscribe to the connection changes.

Additional context
This should open a smooth path to work on #19

@stellasia
Copy link
Owner

Hi @wagnerjt , I've added redux a few weeks ago, but hooks and context should work as well and I'm still ok with removing the dependency to redux. I'll try to explore both options while trying to implement real tests.

This was referenced Aug 10, 2020
@okjodom
Copy link
Contributor

okjodom commented Sep 6, 2020

Pending: Move Layer component to react hooks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants