Add routes with params

[metnew]

Add routes with params

[Metnew]

Hi, thanks for your feedback, I really appreciate it :) 😈
I'll push fix for your issue in next few hours.

[Metnew]

it must be fixed in 23906c6.

[neslinesli93]

Thanks! Although I achieved a similar solution by directly using the path-to-regexp module... What I did not get is how am I supposed to get the matched params of the URL in state.props.match, inside the rendered container

…

On Jun 13, 2017 20:55, "Vladimir Metnew" ***@***.***> wrote: it must be fixed in 23906c6 <23906c6> . — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#14 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABD527gODVxCqeN_O8iQp4MzzQcPwGzvks5sDtsEgaJpZM4N4TFW> .

[Metnew]

Hm, and standard access to match props doesn't work?
https://reacttraining.com/react-router/web/example/url-params

[Metnew]

Seems strange, I'll add one more route with params to app soon, maybe even today.

[neslinesli93]

No wait, I was using a match object defined inside mapStateToProps, instead of directly accessing this.props...

[Metnew]

:(
Is issue resolved now?

[neslinesli93]

Yup, thanks for your help. However I am still wondering how to correctly handle location_change action inside the reducers, having the case of an URL with params inside

…

On Jun 14, 2017 11:33, "Vladimir Metnew" ***@***.***> wrote: :( Is issue resolved now? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#14 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABD527Bba8BnNfpgNRSHh7aiWZYHhpoMks5sD6jngaJpZM4N4TFW> .

[Metnew]

With react-router-redux
IIRC, you can find LOCATION_CHANGE in src/actions/common, in action.payload you can find params and other location stuff