Comments (35)
Will be great to have this project join rackt
!
react-router-redux
sounds good to me 👍
from react-router-redux.
That's great! 👍 💯
from react-router-redux.
good enough 🌟
from react-router-redux.
👍
from react-router-redux.
Sounds good! Just have to make sure it's not too confusing to have two react/redux routers that sound almost the same too. (redux-router
vs react-router-redux
) That could be solved with documentation though. (explaining the differences between the two)
from react-router-redux.
I reckon go for it! Sounds exciting.
from react-router-redux.
I'm going to set a date for this: Wednesday. I'll think through anything we need to do before then.
from react-router-redux.
I stalled on this because I was worried about the rename, sync redux-router is also in rackt it seems like calling this react-router-redux is going to be super confusing.
@ryanflorence @mjackson Are you all sure you want to rename this? If so, I feel like we should either rename redux-router or do something so that it's not so confusing which is which.
from react-router-redux.
Any reason to switch away from redux-simple-router? I usually cringe when things have "simple" in the name (argh SOAP!), but it actually seems warranted in this case
from react-router-redux.
I think Ryan and Michael want it to be more "official" bindings. But I suspect with enough community feedback they might be fine keeping the name.
from react-router-redux.
Maybe we can find a different name for redux-router
too?
from react-router-redux.
We're removing the current redux-router from rackt, so there shouldn't be any confusion.
from react-router-redux.
@acdlite can you transfer it to your account?
from react-router-redux.
Done! :)
from react-router-redux.
I also added a comment to the README directing people to this repo: https://github.com/acdlite/redux-router#for-a-more-stable-official-binding-between-redux-and-react-router-try-redux-simple-router
from react-router-redux.
Thanks for all your work @acdlite!
from react-router-redux.
I just moved it, but I lost access to any settings. @ryanflorence can you help me figure out permissions? github is so confusing with permissions. How can I maintain admin access, as well as give it to kjbekkelund?
I can't rename the repo until I get access back.
from react-router-redux.
react-router-redux
looks great to me... same as react-router
but suffixed with redux.. Looking forward to this!
from react-router-redux.
Correct me if I'm wrong, but redux-simple-router
just ties redux
to a history
object, and isn't actually react-specific. If so, react probably shouldn't be in the new name.
From a loose coupling perspective, you could imagine using this with a redux-based application without react-router at all (i.e. either a completely different non-react router, or perhaps a different react-specific router), as long as those routers were designed to interact with a history
object.
from react-router-redux.
@naw Yep, as of right now we're not coupled to react-router
at all, but we might end up going that route (see #95 — long discussion with lots of thoughts and ideas).
But yeah, either way, I also prefer redux-simple-router
to react-router-redux
.
from react-router-redux.
"Simple", "smart", or "tiny" always end up being lies or offenses. Eventually, your library becomes complex, surprising, or gigantic. If you're lucky, it simply remains an affront (purposeful or not) to other people's attempts at solving the same problem. I think the new name avoids both problems.
My personal interest in bringing it into Rackt is for an official answer to the question "how should I connect react router and redux?". I'd rather a library be named to answer a question v. explaining the (initial) complexity of implementation :)
from react-router-redux.
What about redux-history
?
(Edit: whoops, #80 (comment) and #95 (comment))
from react-router-redux.
Why not just use redux-react-router
?
from react-router-redux.
@jlongster Have you made any decisions on this? Half of this issue was the move to rackt, which is completed. The other half is a rename, which seems to be less pressing (and would nullify the existing "brand" of the project). I'm inclined to say we're done here unless there's a strong desire to rename things.
from react-router-redux.
This project has already evolved to be a little more complex, nothing crazy but I agree with @ryanflorence that the current name isn't very suitable to long-term decisions. Having simple
made a strong point in the beginning but react-router-redux
makes more sense.
We probably should have done this with 2.0.0. Why don't we do this in the next few days with a minor release?
from react-router-redux.
Sure, sounds great!
I say at some point in the near future (this weekend?) we push a 2.1.0 release under the new name and then deprecate 2.0.4 under the old name with a warning about the name switch. I also say we keep releasing under both names until 3.0 hits (if ever), deprecating any releases under redux-simple-router
with that same warning. Sound like a plan?
from react-router-redux.
@timdorr Sounds good, but I don't think we need to keep releasing under both names. I heard from Dan Abramov you can tell npm to redirect it, so npm install redux-simple-router
still works even if our package is react-router-redux
. If that works, the biggest pain point goes away (both github and npm will redirect).
I might be remembering wrong... hope not.
from react-router-redux.
It sounds like they can maybe do it for us, but it might be delayed or ignored: npm/npm#7515 I think npm deprecate
is the preferred option for now.
from react-router-redux.
No, I think only GitHub repos can be redirected. With npm you need to deprecate.
from react-router-redux.
Oh, that's right. Isn't there at least a way to tell the user "this is deprecated, you should download react-router-redux instead"? I could have sworn there was, but I'm probably wrong. I'll look into it more today/tomorrow.
I'm not sure I want to keep releasing under both names, that seems like it could be more confusing, and just make people change it to get the new version. It shouldn't be too hard to change names, just find/replace.
from react-router-redux.
Yes, that's what npm deprecate
does.
from react-router-redux.
We need to push a 2.1.0 for #218 and #225. That would be a good time to do a rename, so I can take care of all of that around the same time, if there are no objections.
from react-router-redux.
Ok, that sounds good.
from react-router-redux.
OK, kicking this off now. I'm going to merge in #218, update the URLs in the project, and then publish that as a 2.1.0 under the react-router-redux
package on NPM. I'll then npm deprecate
2.0.4 and we'll be good to go.
from react-router-redux.
from react-router-redux.
Related Issues (20)
- with react-router 4.X and react-router-redux 5.0, the devTools can not replay different path HOT 3
- Expose defaultSelectLocationState HOT 2
- Action name HOT 2
- about goBack and go(-1) HOT 1
- Error: Element type is invalid: expected a string HOT 1
- example change url but not component HOT 1
- why browserHistory not working?
- Why there is no action creator for LOCATION_CHANGE? HOT 2
- routes not working properly HOT 1
- how to use hashRouter in react-router-redux HOT 1
- push() replaces url HOT 4
- History with other sites. HOT 1
- basename not working on react-router-redux 5.0.0-alpha.9 HOT 1
- Where is `this.props.location` and `this.props.history` in react-router-redux? HOT 1
- How does `selectLocationState` work? Give examples HOT 3
- LOCATION_CHANGE is wrong action type HOT 1
- How to preserve router in ownProps? HOT 1
- -router
- Ability to modify 'location' with my own reducer? HOT 1
- The readme section "We're moving" is misleading. The "new" location says *Project Deprecated* HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from react-router-redux.