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

🐛 manual invoke single-spa start function for parcels and set urlRerouteOnly true as default #1414

Merged
merged 1 commit into from
Apr 26, 2021

Conversation

@kuitos kuitos requested a review from gongshun April 24, 2021 14:45
Copy link
Collaborator

@gongshun gongshun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

文档是否需要补充一下

@kuitos
Copy link
Member Author

kuitos commented Apr 26, 2021

文档是否需要补充一下

你指哪一块的文档?urlRerouteOnly 这个吗?

@gongshun
Copy link
Collaborator

你指哪一块的文档?urlRerouteOnly 这个吗?

是的

@kuitos
Copy link
Member Author

kuitos commented Apr 26, 2021

这个感觉不用透出了,如果默认值不会对以前的行为造成影响的话,那用户也不需要感知到这个配置

@ycpaladin
Copy link

angular v11 主应用里面引用2.4.1,会导致在主应用里面跳转(使用routerLink指令)子应用(angular v11)无效,原因就是 2.4.1里面的 start() 方法中默认 {urlRerouteOnly: true} 了。

@gongshun
Copy link
Collaborator

angular v11 主应用里面引用2.4.1,会导致在主应用里面跳转(使用routerLink指令)子应用(angular v11)无效,原因就是 2.4.1里面的 start() 方法中默认 {urlRerouteOnly: true} 了。

大部分情况是不需要 urlRerouteOnly: true 的,你如果需要的话, start 的时候是可以配置的

@ycpaladin
Copy link

ycpaladin commented May 18, 2021

angular v11 主应用里面引用2.4.1,会导致在主应用里面跳转(使用routerLink指令)子应用(angular v11)无效,原因就是 2.4.1里面的 start() 方法中默认 {urlRerouteOnly: true} 了。

大部分情况是不需要 urlRerouteOnly: true 的,你如果需要的话, start 的时候是可以配置的

是的,在angular v11环境中升级到2.4.1需要 start({urlRerouteOnly: false});

既然大部分情况是不需要 urlRerouteOnly: true 的, 为什么2.4.1中 urlRerouteOnly 默认值为true?

@kuitos
Copy link
Member Author

kuitos commented May 18, 2021

大部分情况是不需要 urlRerouteOnly: true 的

应该是大部分情况不需要设置 urlRerouteOnly: false 吧? @gongshun

@gongshun
Copy link
Collaborator

应该是大部分情况不需要设置 urlRerouteOnly: false 吧? @gongshun

对,说错了。

@wangi4myself
Copy link

#1071
qiankun 2.6.3的版本 我在路由页面手动加载start。首页进入还是会有同样的抛错

kuitos added a commit that referenced this pull request May 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment