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

change routeFactory to routeMap #2152

Open
wants to merge 1 commit into
base: dev/exp
Choose a base branch
from

Conversation

joechan-cq
Copy link
Collaborator

主要解决routeFactory会多次调用的问题,将初始化的routeFactory方法改为routeMap,原有构造方法废弃,作为升级版本后的兼容方案予以保留。

fix #2122fix #1869fix #1599

将初始化的routeFactory方法改为routeMap
@joechan-cq joechan-cq requested a review from 0xZOne November 19, 2024 01:46
@niegaotao
Copy link

niegaotao commented Dec 25, 2024

直接新增入参List? routeNames = null是不是更好?

@joechan-cq
Copy link
Collaborator Author

多传一个参数就会要求使用者外部自行同步两个参数的内容,可能会出问题。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants