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

There are some bugs in the Console UI #8784

Closed
onewe opened this issue Jul 22, 2022 · 0 comments · Fixed by #8787
Closed

There are some bugs in the Console UI #8784

onewe opened this issue Jul 22, 2022 · 0 comments · Fixed by #8787

Comments

@onewe
Copy link
Collaborator

onewe commented Jul 22, 2022

Console ul can't show the error tip message correctly
How to reproduce:

  1. enable auth
  2. create an account on the console ul without any permission
  3. login the console

Fix:

  1. add properties server.error.include-message=ALWAYS in the application.properties

控制台不能正确显示提示信息
复现步骤:

  1. 开启鉴权
  2. 控制台创建一个账号,不做赋权操作
  3. 登录控制台,无法正确提示错误信息

修复:

  1. 后端增加 server.error.include-message=ALWAYS 配置

Can't reset user password
How to reproduce:

  1. create an account on the console ul
  2. use nacos user login and jump to the user list page
  3. click modify button, it can't show the modify dialog and nothing happen

fix:

  1. add visible field at the PasswordReset component

不能重置密码
复现步骤:

  1. 在控制台创建一个账号
  2. 使用 nacos 用户登录并跳转到用户列表页面
  3. 点击修改按钮,不能显示修改对话框没有任何反应

修复:

  1. PasswordReset 组建增加 visible 字段

Access-Control-Allow-Origin
How to reproduce:

  1. enter the console-ul directory and run yarn start
  2. open the home page on the chrome

fix:

  1. use addAllowedOriginPattern instead of addAllowedOrigin

跨域
复现步骤:

  1. 进入 console-ul 目录并运行 yarn start
  2. 在 chrome 浏览器中打开页面

修复:

  1. 使用 addAllowedOriginPattern 替换 addAllowedOrigin
@onewe onewe changed the title Console ul can't show the error tip message correctly Console ul can't show the error tip message correctly and some bugs Jul 22, 2022
onewe added a commit to onewe/nacos that referenced this issue Jul 22, 2022
- add server.error.include-message property
- add no permission tip when push config
- use addAllowedOriginPattern instead of addAllowedOrigin
- remove RAM logic in the lib.js
- fix button type error and mode value error in NewConfig.js
- fix the password reset dialog can't show

Close alibaba#8784
@onewe onewe changed the title Console ul can't show the error tip message correctly and some bugs Console ul can't show the error tip message correctly and there is some bugs Jul 22, 2022
@onewe onewe changed the title Console ul can't show the error tip message correctly and there is some bugs there are some bugs in the Console UI Jul 22, 2022
@onewe onewe changed the title there are some bugs in the Console UI There are some bugs in the Console UI Jul 22, 2022
onewe added a commit to onewe/nacos that referenced this issue Jul 22, 2022
onewe added a commit to onewe/nacos that referenced this issue Jul 22, 2022
onewe added a commit to onewe/nacos that referenced this issue Jul 22, 2022
KomachiSion pushed a commit that referenced this issue Jul 25, 2022
* [ISSUE #8784] fix some bugs for Console UI

- add server.error.include-message property
- add no permission tip when push config
- use addAllowedOriginPattern instead of addAllowedOrigin
- remove RAM logic in the lib.js
- fix button type error and mode value error in NewConfig.js
- fix the password reset dialog can't show

Close #8784

* [ISSUE #8784] fix translate error

* [ISSUE #8784] user ON_PARAM instead of ALWAYS

* [ISSUE #8784] fix user reset password in the Header.js
beet233 pushed a commit to beet233/nacos that referenced this issue Jul 25, 2022
* [ISSUE alibaba#8784] fix some bugs for Console UI

- add server.error.include-message property
- add no permission tip when push config
- use addAllowedOriginPattern instead of addAllowedOrigin
- remove RAM logic in the lib.js
- fix button type error and mode value error in NewConfig.js
- fix the password reset dialog can't show

Close alibaba#8784

* [ISSUE alibaba#8784] fix translate error

* [ISSUE alibaba#8784] user ON_PARAM instead of ALWAYS

* [ISSUE alibaba#8784] fix user reset password in the Header.js
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 a pull request may close this issue.

1 participant