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

multiple loads route #1202

Merged
merged 2 commits into from
Jul 22, 2021
Merged

multiple loads route #1202

merged 2 commits into from
Jul 22, 2021

Conversation

MrAccle
Copy link
Contributor

@MrAccle MrAccle commented Jun 24, 2021

First of all, thank you for your contribution! 😄

Pull request will be merged after one of collaborators approve.
Please makes sure that these form are filled before submitting your pull request, thank you!

这个变动的性质是

  • 新特性提交
  • 日常 bug 修复
  • 文档改进
  • 组件样式改进
  • 重构
  • 代码风格优化
  • 分支合并
  • 其他改动(是关于什么的改动?)

需求背景

问题描述:
在多次进行动态权限路由的添加时,如果前一次对路由进行过滤后,若不整页刷新,第二次动态路由过滤时的路由对象为第一次过滤后的路由对象,从而可能导致第二次需要第一次过滤掉路由时,无法正常显示,只能通过页面刷新从而恢复正常。

实现方案和 API(非新功能可选)

  1. 在 src/store/modules/permission.js 里将路由重新深拷贝后在进行权限过滤

对用户的影响和可能的风险(非新功能可选)

  1. 仅在静态路由文件时生效;服务端获取的动态路由可以忽视此问题。

Changelog 描述(非新功能可选)

  1. multiple loads route

请求合并前的自查清单

  • 文档已补充或无须补充
  • 代码演示已提供或无须提供
  • Changelog 已提供或无须提供

后续计划(非新功能可选)

@sendya
Copy link
Member

sendya commented Jul 13, 2021

ping ... ?

@MrAccle
Copy link
Contributor Author

MrAccle commented Jul 16, 2021

Thanks for your advice,It has been revised.

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