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

组织管理-组织新增用户不正确提示 #66

Closed
marsdawe opened this issue Mar 7, 2023 · 7 comments · Fixed by bestchains/bc-apis#57
Closed

组织管理-组织新增用户不正确提示 #66

marsdawe opened this issue Mar 7, 2023 · 7 comments · Fixed by bestchains/bc-apis#57
Assignees
@dayuy @Jandiasnow @marsdawe
Labels
bug Something isn't working medium Medium priority
Milestone
v0.2

Comments

@marsdawe
Copy link

marsdawe commented Mar 7, 2023 •
edited
Loading

  1. 组织新增用户时，如果输入的用户不存在，仍提示邀请成功，应提示用户不存在！
  2. 情况1出现之后，组织状态变为“异常”
@marsdawe
Copy link
Author

marsdawe commented Mar 7, 2023

image

@Jandiasnow
Copy link
Member

需要提供校验接口，或提供报错信息 @dayuy

@bjwswang
Copy link
Member

@marsdawe 后端也增加校验

@dayuy
Copy link

dayuy commented Mar 17, 2023

建议：1. 这里是不是可以搞成多选框，选择已有成员（可搜索）@marsdawe
2. bff 会加校验

@bjwswang bjwswang added bug Something isn't working medium Medium priority labels Mar 17, 2023
@bjwswang bjwswang added this to the v0.2 milestone Mar 17, 2023
@bjwswang
Copy link
Member

@marsdawe 后端也增加校验

后端已经添加对用户是否存在的校验

@dayuy
Copy link

dayuy commented Mar 20, 2023

bff 已加校验

@dayuy dayuy mentioned this issue Mar 20, 2023
Merged
@marsdawe
Copy link
Author

marsdawe commented Apr 7, 2023

image
应提示“用户不存在”

@dayuy dayuy mentioned this issue Apr 7, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dayuy dayuy

@Jandiasnow Jandiasnow

@marsdawe marsdawe

Labels
bug Something isn't working medium Medium priority
Projects
None yet
Milestone
v0.2
Development

Successfully merging a pull request may close this issue.

fix: add spec.externalBuilder when creating chaincode dayuy/bc-apis
4 participants
@dayuy @Jandiasnow @bjwswang @marsdawe