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

upgrade to react-router 4? #47

Open
davidhu2000 opened this issue Apr 18, 2017 · 5 comments
Open

upgrade to react-router 4? #47

davidhu2000 opened this issue Apr 18, 2017 · 5 comments

Comments

@davidhu2000
Copy link
Owner

So React Router 4 is the current stable version, there seems to be a ton of changes. For example, hashHistory is no longer defined. But there seem to be some good features.

I want to ask you guys if you think we should migrate from v3 to v4? I think we probably eventually have to do it, but if it's done earlier, it might be better?

What do you think?

Here is the doc for v4
https://reacttraining.com/react-router/

@nguyenkevin16 @asherman-ca @rlee0525 @cjudge1337

@cjudge1337
Copy link
Collaborator

I don't see why not. It looks like all we would really need to do at this point is change hashHistory to history everywhere. Also, we could potentially speed the app up a little by switching to <MemoryRouter> from <Router>.

@davidhu2000
Copy link
Owner Author

What is the difference between <MemoryRouter> and <Router> ? @cjudge1337

@cjudge1337
Copy link
Collaborator

cjudge1337 commented Apr 19, 2017

Just going off the description they had for it right now:

A <Router> that keeps the history of your “URL” in memory (does not read or write to the address bar). Useful in tests and non-browser environments like React Native.

I don't know if it would actually help but since we are doing a desktop app this seems like the kind of environment it would be useful in.

@davidhu2000
Copy link
Owner Author

are you interested in looking into possible migrating our app to v4?

Or at least to see how feasible it is to do it? Or whether it is worth the effort? @cjudge1337

@cjudge1337
Copy link
Collaborator

Sure I can take a look.

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

No branches or pull requests

2 participants