neilff / redux-ui-router

ngRedux bindings for Angular UI Router
MIT License
143 stars 48 forks source link

Added stateChangeFinish action to ensure state changes are set in the… #80

Closed hally9k closed 7 years ago

hally9k commented 7 years ago

… app state before components are instantiated

hally9k commented 7 years ago

In reaction to @Nighthawk14 comment on the previous PR for this branch - Master is currently the version that we will publish as 0.7.0 when ui-router goes 1.0. But I take your point. Maybe master should represent our current stable release? I have made a release/0.7.0 branch to merge these changes into for now. Then once ui-router 1.0s we can release 0.7.0 and merge into master. Does that sound like the correct approach?

Nighthawk14 commented 7 years ago

Yep that looks good! Always having master as the stable version working with the current stable version of ui-router is a good approach for me. You can have beta releases from branches for this kind of stuff 👍