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

[Bugfix]修复权限ACL管理中,消费组列表展示错误的问题(#991) #1037

Merged
merged 1 commit into from
Jun 25, 2023

Conversation

HZSUZJ
Copy link
Contributor

@HZSUZJ HZSUZJ commented Jun 5, 2023

请不要在没有先创建Issue的情况下创建Pull Request。

变更的目的是什么

修复权限ACL管理中,消费组列表展示错误的问题

简短的更新日志

1.消费组列表里面展示的是Group列表
2.并像生产组一样判断Group是否存在

验证这一变化

消费组:
image
代码修改后效果图:
image
image

请遵循此清单,以帮助我们快速轻松地整合您的贡献:

  • 一个 PR(Pull Request的简写)只解决一个问题,禁止一个 PR 解决多个问题;
  • 确保 PR 有对应的 Issue(通常在您开始处理之前创建),除非是书写错误之类的琐碎更改不需要 Issue ;
  • 格式化 PR 及 Commit-Log 的标题及内容,例如 [Bugfix]修复用户信息修改后,用户列表会抛出空指针异常的问题(#860) #861 。PS:Commit-Log 需要在 Git Commit 代码时进行填写,在 GitHub 上修改不了;
  • 编写足够详细的 PR 描述,以了解 PR 的作用、方式和原因;
  • 编写必要的单元测试来验证您的逻辑更正。如果提交了新功能或重大更改,请记住在 test 模块中添加 integration-test;
  • 确保编译通过,集成测试通过;

@ZQKC
Copy link
Collaborator

ZQKC commented Jun 6, 2023

这个地方需要允许为不存在的Group添加ACL权限,但是看那个效果图,是一定需要Group存在的,这个地方需要修改一下。

允许为不存在的Group添加ACL权限的原因:
1、添加Group前缀匹配的权限时,所填写的Group往往不存在;
2、也可能出现Group还没有,但是想提前增加权限的动作;

@HZSUZJ
Copy link
Contributor Author

HZSUZJ commented Jun 6, 2023

这个地方需要允许为不存在的Group添加ACL权限,但是看那个效果图,是一定需要Group存在的,这个地方需要修改一下。

允许为不存在的Group添加ACL权限的原因: 1、添加Group前缀匹配的权限时,所填写的Group往往不存在; 2、也可能出现Group还没有,但是想提前增加权限的动作;

修改完毕,将验证部分关闭。

@HZSUZJ HZSUZJ reopened this Jun 6, 2023
@HZSUZJ HZSUZJ closed this Jun 6, 2023
@HZSUZJ HZSUZJ reopened this Jun 6, 2023
@ZQKC ZQKC linked an issue Jun 19, 2023 that may be closed by this pull request
@lucasun lucasun merged commit 3f518c9 into didi:master Jun 25, 2023
@ZQKC ZQKC added this to the v3.4.0 milestone Dec 3, 2023
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.

权限ACL管理中,消费组列表展示错误
3 participants