chatgpt-web-dev / chatgpt-web

A third-party ChatGPT Web UI page built with Express and Vue3, through the official OpenAI completion API. / 用 Express 和 Vue3 搭建的第三方 ChatGPT 前端页面, 基于 OpenAI 官方 completion API.
https://chatgpt-web.dev
MIT License
1.56k stars 436 forks source link

为新注册用户设置全局初始次数,并且同步全局次数限制的开关 #478

Closed ShenKeeeew closed 3 months ago

Kerwin1202 commented 4 months ago
  1. 能不能把这个输入框 照着下面这 用户修改里面的 这种 number的 而不是提示 避免输入错误

    image

    改为下面这种 右侧可以直接加碱 组件自动不允许数字以外的

    image
  2. 我刚测试打开之后,发现 默认去修改 单独用户的 里面都是 false 但是应该是开启的? 如上图 这种逻辑 我觉得 其实应该是 有几个不同状态的, 如果 默认关闭 所有用户没动过的情况下 都是关闭的 如果开启也是同理 但是如果 关闭的时候 某个用户 单独开启了, 这之后 去开启 那么所有用户都开启了,但是 再关闭了,这时候 刚刚单独开启的用户状态是否应该也关闭呢还是保持开启,数据库的状态是否要进行响应的更改,如果直接 update 我觉得 有点多 是否直接 unset 字段去掉 然后读取全局的 这块没有提现出来

  3. 麻烦格式化一下哈 看到好几个都是前面 多了空格

image
BobDu commented 4 months ago

我刚测试打开之后,发现 默认去修改 单独用户的 里面都是 false 但是应该是开启的? 如上图 这种逻辑 我觉得 其实应该是 有几个不同状态的, 如果 默认关闭 所有用户没动过的情况下 都是关闭的 如果开启也是同理 但是如果 关闭的时候 某个用户 单独开启了, 这之后 去开启 那么所有用户都开启了,但是 再关闭了,这时候 刚刚单独开启的用户状态是否应该也关闭呢还是保持开启,数据库的状态是否要进行响应的更改,如果直接 update 我觉得 有点多 是否直接 unset 字段去掉 然后读取全局的 这块没有提现出来

在新用户默认限制次数配置项外 再新增一个新用户是否开启用户级别的次数限制的配置项?

以上两个配置 只有在打开全局次数限制开关的前提下。才能进行修改。 (因为现在的逻辑就是。只有在开启了全局的次数限制功能之后。用户才能看到自己的剩余次数以及使用兑换码等)

然后对这两个配置项的修改只影响到新用户 已经注册完成的用户不受影响。

如果按这个流程方案是否会更清晰一些?

ShenKeeeew commented 4 months ago
  1. 本来计划使用NInputNumber 修改的,但是不知道为啥写成下面这样,数据不会被post,所以就这样将就写了。(抱歉,没学过typescript,请大佬赐教怎么修改。)
    
    <NInputNumber
              :value="config && config.globalAmount"
              :autosize="{ minRows: 1, maxRows: 2 }" placeholder=""
       />


2. 如 @BobDu ,目前这个次数限制的开关和初始次数,只对新用户生效。开关的变动和初始次数数值的变动,不对现有用户造成任何影响。

3. 格式化完成了。
BobDu commented 4 months ago

本来计划使用 NInputNumber 修改的

修改成了使用 NInputNumber 需要使用 v-model:value 而不能使用 :value 可以看下是否符合需求 ping @ShenKeeeew

ShenKeeeew commented 4 months ago

可以了,感谢指点

BobDu commented 4 months ago

是否可以再新增一个 新用户是否在注册时自动开启用户级别次数限制 的配置项? @ShenKeeeew

ShenKeeeew commented 4 months ago

使用次数限制 这个开关目前已经达到这个目的了吧?开启开关时,新用户注册就会自动开启用户的次数限制,关闭之后新用户就不会开启。

BobDu commented 4 months ago

抱歉因为个人原因回复较晚。可否有一个稳定的分支用于等待merge?我看又继续进行了三次提交。

ShenKeeeew commented 4 months ago

新的三次提交是在后台设置公告内容和开关(如下面的截图),用来在主页面进行弹窗提醒,我测试了没有问题就合并到主分支了,没想到会影响到这里(不好意思,git不熟练,我要回退一下吗) 2282880F-D587-47F2-AA92-970D2B68E3CF F55731A3-4A36-4CF9-9F17-61CAF88AC825

BobDu commented 4 months ago

是的。辛苦修改 main 分支 (你用于发起PR的分支) 的 HEAD 修改到进行这三次提交之前。再 force push一下哈。

操作前记得再单独的分支备份你的这个修改哈。 以免永久性丢失代码。(一般来说在操作前先在main上切出一个类似 backup 的分支把这个分支push之后。再checkout回main操作就能实现备份。当前具体操作每个人习惯不同都可以哈。只是提醒一下避免你丢代码)

dqzboy commented 4 months ago

新的三次提交是在后台设置公告内容和开关(如下面的截图),用来在主页面进行弹窗提醒,我测试了没有问题就合并到主分支了,没想到会影响到这里(不好意思,git不熟练,我要回退一下吗) 2282880F-D587-47F2-AA92-970D2B68E3CF F55731A3-4A36-4CF9-9F17-61CAF88AC825

大佬厉害,这个弹窗效果不错👍

ShenKeeeew commented 3 months ago

是的。辛苦修改 main 分支 (你用于发起PR的分支) 的 HEAD 修改到进行这三次提交之前。再 force push一下哈。

操作前记得再单独的分支备份你的这个修改哈。 以免永久性丢失代码。(一般来说在操作前先在main上切出一个类似 backup 的分支把这个分支push之后。再checkout回main操作就能实现备份。当前具体操作每个人习惯不同都可以哈。只是提醒一下避免你丢代码)

已经退回了,现在是之前的状态了,这部分合并完,我再提弹公告的pr,谢谢