feat: 修改验证账号逻辑，需要管理员开通，修复一些bug #6
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
而且再加个配置选项? 毕竟我感觉不是所有人都需要，有需要的开启了选项管理员才可以收到，而且另一方面我觉得账号申请开启了之后 注册者 应该就收不到邮件，就类似那种论坛的 虽然开了注册 但是自己注册是不行的 管理员审核才可以 |
|
我的逻辑是，注册者先验证邮箱，验证成功后，发邮件通知管理，管理再点激活，最后通知注册者。选项先不加了，代码判断有点多了。这个能注册就能用，等于全开放，不是我想要的，但同步记录是需要账号，这个是没错的，其实更合理的做法是加用户管理，然而这只是我个人使用的一个小工具，没有更多的想法… |
|
👍 |
|
其实挺好的，我本来想合并 然后我来加个判断的。。结果 force push 掉了，有时候确实 注册放开 需要确认 |
|
我每天都会同步你的和chan的代码，你可以下载代码用beyondcompare对比下，也没加啥内容，主要修改些小问题，只是个人使用的，加个审核比较稳，或者加个动态暗号啥的，最理想还是加用户管理，没时间弄，这vue3也不熟悉，后端简单，前端的话，chan架构得有点复杂了，不够简单明了，我不太喜欢这种风格，而且也没有注释，没空研究那么多了，哈哈。现在不好push了，可能会有很多冲突，我在合并你们的代码时，都要手改，不能自动合并。 |
Kerwin1202
pushed a commit
that referenced
this pull request
Apr 5, 2023
|
已合并 |
|
非常赞同这个逻辑，如果楼主已经写好代码，希望合并或者看一下你的分支，谢谢 |
|
唉哟，不错哦，发现个更好的前端，有空可以看看：https://github.com/ztjhz/BetterChatGPT |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
如题，有需要的就合并下。