Call 'data' hook by router.go() force option and router.reload() #442
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add jspm configuration
Remove cjs style exports from pipeline
更新了 Lazy Loading Routes 功能介绍
优化lazy load语言表达
Minor tweaks for Japanese docs
Avoid an error when a root '/' for VueRouter is specified (fix vuejs#379)
expose router.stringifyPath (fix vuejs#409)
docs: replace v-repeat to v-for
Fix delegate v-link when router has root configuration
The original code would encode full path differently based on whether the path was supplied as `name` or as `path`. Fixed so that all branches conform to the same encoding order, which is encoding of URI components and then or URI itself. Also, route-recognizer would decode in inverse order (first decode URI components and then URI). That was fixed as well (encode and decode orders should be inverse).
|
You need to rebase this branch |
|
Will be great if this branch is rebase and this feature integrated. |
|
I try this by patching my router manullay and actually the pipeline is activated and the transitionOut is playing but then there is something blocking in the canDeactivate and the pipeline is stop.. :( |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Having router reload function (just call
datahook for same component) would be a nice feature from several reported issues #296 etc.force: trueoption forrouter.go()The
datahook will be called again even when a current same URL is specified.Example:
this.$router.go({ path: '/a', force: true })from components<a v-link="{ path: '/inbox/message/123', force: true }">inbox</a>will triggerdatahook by every clickrouter.reload()