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

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

Merged
merged 5 commits into from
Mar 20, 2024

Conversation

ShenKeeeew
Copy link
Contributor

No description provided.

@Kerwin1202
Copy link
Member

  1. 能不能把这个输入框 照着下面这 用户修改里面的 这种 number的 而不是提示 避免输入错误
image 改为下面这种 右侧可以直接加碱 组件自动不允许数字以外的 image
  1. 我刚测试打开之后,发现 默认去修改 单独用户的 里面都是 false 但是应该是开启的? 如上图
    这种逻辑 我觉得 其实应该是 有几个不同状态的,
    如果 默认关闭 所有用户没动过的情况下 都是关闭的 如果开启也是同理
    但是如果 关闭的时候 某个用户 单独开启了, 这之后 去开启 那么所有用户都开启了,但是 再关闭了,这时候 刚刚单独开启的用户状态是否应该也关闭呢还是保持开启,数据库的状态是否要进行响应的更改,如果直接 update 我觉得 有点多 是否直接 unset 字段去掉 然后读取全局的 这块没有提现出来

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

image

@BobDu
Copy link
Member

BobDu commented Mar 15, 2024

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

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

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

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

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

@ShenKeeeew
Copy link
Contributor Author

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

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

  2. 格式化完成了。

Signed-off-by: Bob Du <i@bobdu.cc>
@BobDu
Copy link
Member

BobDu commented Mar 15, 2024

本来计划使用 NInputNumber 修改的

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

@ShenKeeeew
Copy link
Contributor Author

可以了,感谢指点

@BobDu
Copy link
Member

BobDu commented Mar 16, 2024

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

@ShenKeeeew
Copy link
Contributor Author

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

@BobDu
Copy link
Member

BobDu commented Mar 18, 2024

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

@ShenKeeeew
Copy link
Contributor Author

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

@BobDu
Copy link
Member

BobDu commented Mar 19, 2024

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

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

@dqzboy
Copy link

dqzboy commented Mar 19, 2024

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

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

@ShenKeeeew ShenKeeeew force-pushed the main branch 2 times, most recently from 0d4583f to 0da0067 Compare March 19, 2024 09:53
@ShenKeeeew
Copy link
Contributor Author

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

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

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

Copy link
Member

@BobDu BobDu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@BobDu BobDu merged commit 7e195d1 into chatgpt-web-dev:main Mar 20, 2024
2 checks passed
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 this pull request may close these issues.

4 participants