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

管理页面系统设置优化建议 #17

Open
Pinkpiglet opened this issue Mar 27, 2023 · 7 comments · Fixed by #19
Open

管理页面系统设置优化建议 #17

Pinkpiglet opened this issue Mar 27, 2023 · 7 comments · Fixed by #19
Labels
bug Something isn't working enhancement New feature or request

Comments

@Pinkpiglet
Copy link

1、黑名单、白名单设置建议将设置的值返回至输入框中。
image
2、apikey脱敏显示在输入框中。
image
3、将管理后台嵌至前台,判断用户是否是管理员,是则显示管理后台图标显示,不是则隐藏。
image
望采纳!

@suikodev
Copy link
Collaborator

  1. 第一点是会的, 进入设置页面会将黑白名单返回至输入框中:

image

没有就说明可能是 bug, 麻烦提供一下更多的信息,比如可以打开浏览器控制台查看是否有错误信息,接口返回是否正常等。
  1. 会优化

  2. 会做

@suikodev suikodev added enhancement New feature or request bug Something isn't working labels Mar 27, 2023
@zzkk74006 zzkk74006 assigned zzkk74006 and unassigned zzkk74006 Mar 27, 2023
suikodev pushed a commit that referenced this issue Mar 27, 2023
chore: 优化 `service` 内命令
@Pinkpiglet
Copy link
Author

Pinkpiglet commented Mar 28, 2023

@zzkk74006 @suikodev @
更新白名单及黑名单,未写入至Datastore,但是前端提示成功。
image
image
image

@HankMo
Copy link

HankMo commented Mar 28, 2023

@Pinkpiglet 第三点我感觉很方便,如果后台功能性不是很复杂,就前置在前端显示就好了,方便一点。如果后台功能很多还是单独方便。还有一点建议就是前端后端能不能部署一次,都用一个端口,我看现在是前端启动一次,然后后端启动一次。能不能简化前后端合并启动一次就行了,这样占用一个端口,也方便一点。(个人拙见)

@Pinkpiglet
Copy link
Author

@HankMo 其实前后端分离挺好的

@Pinkpiglet
Copy link
Author

@suikodev @zzkk74006 第一和第二个问题是我操作问题。

@suikodev
Copy link
Collaborator

@HankMo

  1. 管理员的页面一直在前端项目内的,只是当时没想好管理员页面的入口放在 chat 页面中的哪里比较合适
  2. 前后端部署一次就是只用一个后端服务的端口的。前端启动一次,后端启动一次是开发时才有的行为。在生产环境使用 node.js 部署这个项目时应该先通过 yarn 安装依赖,然后通过 yarn build 构建出静态文件文件夹 dist ,然后将 dist 重命名为 public 放到 service 文件夹中,在 service 文件夹中再通过 yarn 安装依赖,再 yarn build 构建一下,最后再用 yarn prod 启动。这样整个项目就只会占用一个 3002 端口

如果还有疑惑,可以随时提问

@suikodev
Copy link
Collaborator

@zzkk74006 想一下第三点

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants