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.

Sign up for GitHub

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

Jump to bottom

【优化】用户入职状态变更为离职应该限制登录使用 #929

Closed
Canway-shiisa opened this issue Feb 22, 2023 · 4 comments
Closed

【优化】用户入职状态变更为离职应该限制登录使用 #929

Canway-shiisa opened this issue Feb 22, 2023 · 4 comments
Assignees
@wklken @nannan00 @Xmandon @yuri0528
Labels
canway 需求来源为嘉为侧
Milestone
Y2023M13

Comments

@Canway-shiisa
Copy link
Collaborator

Canway-shiisa commented Feb 22, 2023 •
edited

存在的问题
目前用户变更入职状态为离职的时候不会同步变更enbaled字段，这就使得离职状态下用户还能进行正常登录及其他一系列操作

重现方法
点击用户 -> 进行编辑 -> 将用户的在职状态字段变更为离职 -> 使用该用户进行登录测试及其他用户管理相关操作
image

理想行为
离职状态的用户不应该能正常登录及进行用户管理的其他操作

版本
当前最新版仍旧存在该问题

额外信息
处理：前端可以在该字段更新为离职的时候 同步变更enabled字段为0

  • 相关接口：
    image

image
@Canway-shiisa Canway-shiisa changed the title 【优化】入职状态变更为离职应该同步变更enabled字段使用户不可用 【优化】【前端】入职状态变更为离职应该同步变更enabled字段使用户不可用 Feb 22, 2023
@Canway-shiisa
Copy link
Collaborator Author

Canway-shiisa commented Feb 24, 2023 •
edited

  1. 这里同步修改enabled可能会影响到用户展示列表，enabled=0的部分默认就不展示了：
    image
  2. 如果变成1.中的展示逻辑，那搜索框中支持在职状态就没有意义了：
    image
  3. 综上，是否不进行enabled的同步变更，而是同步变更账号状态字段为禁用=》即status=DISABLED
  4. 存在一种情况：用户状态变更为离职->同步变更账号状态为禁用->管理员手动重新启用用户->这时候也会出现离职用户可以进行登录及其他一切正常操作的情况，这里是否需要增加提示

@wklken
Copy link
Collaborator

wklken commented Feb 24, 2023

登录的检测逻辑直接多加一条状态必须在职? 如果离职直接禁止登录

其他地方的逻辑不变

@Shutulee Shutulee added this to the Y2023M09 milestone Feb 27, 2023
@nannan00
Copy link
Collaborator

nannan00 commented Mar 3, 2023 •
edited

登录的检测逻辑直接多加一条状态必须在职? 如果离职直接禁止登录

其他地方的逻辑不变

+1

用户管理只关注数据自身，不应该关注数据被用于什么场景，然后需要联合变更其他数据

离职不能登录，这个是登录的场景，应该是 登录消费用户管理的数据，即登录时，判断用户的离职状态

需要调整登录的逻辑，支持判断离职状态并提醒具体登录失败原因

@Canway-shiisa Canway-shiisa modified the milestones: Y2023M09, Y2023M10 Mar 6, 2023
@Canway-shiisa Canway-shiisa modified the milestones: Y2023M10, Y2023M11 Mar 13, 2023
@Canway-shiisa Canway-shiisa changed the title 【优化】【前端】入职状态变更为离职应该同步变更enabled字段使用户不可用 【优化】用户入职状态变更为离职应该限制登录使用 Mar 21, 2023
Canway-shiisa added a commit to Canway-shiisa/bk-user that referenced this issue Mar 21, 2023
@Canway-shiisa
feat:check staff status before login TencentBlueKing#929
3c0f13c
@Canway-shiisa Canway-shiisa mentioned this issue Mar 24, 2023
Merged
@nannan00 nannan00 modified the milestones: Y2023M11, Y2023M13 Mar 27, 2023
nannan00 pushed a commit that referenced this issue Mar 27, 2023
@Canway-shiisa
feat:check staff status before login #929 (#945)
705515d
@Canway-shiisa Canway-shiisa added the canway 需求来源为嘉为侧 label Mar 28, 2023
@Canway-shiisa
Copy link
Collaborator Author

v2.5.3中实现：登录校验用户在职状态，离职用户限制登录

@nannan00 nannan00 mentioned this issue Apr 7, 2023
Merged
yuri0528 pushed a commit to yuri0528/bk-user that referenced this issue May 8, 2023
@Canway-shiisa @yuri0528
feat:check staff status before login TencentBlueKing#929 (TencentBlue…
805c15f 
…King#945)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wklken wklken

@nannan00 nannan00

@Xmandon Xmandon

@yuri0528 yuri0528

Labels
canway 需求来源为嘉为侧
Projects
None yet
Milestone
Y2023M13
Development

No branches or pull requests
6 participants
@wklken @nannan00 @Xmandon @Shutulee @yuri0528 @Canway-shiisa