Valine-Admin icon indicating copy to clipboard operation
Valine-Admin copied to clipboard

在不影响原功能代码的情况下,增加了一些本机,VPS部署,或者docker构建的一些文件

Open iouAkira opened this issue 5 years ago • 3 comments

  • send-notification.js 这个文件主要根据repo 里面 ./utilities/send-mail.jscloud.js 里面提取出来改造成一个专门给crontab 做定时任务发送通知邮件用的。因为这个文件单独执行所以,没有原Comment表的更新权限,所以这里单独将发送通知记录到一张新的Notifications表。
  • ./routes/comment.js 这文件里面增加了 updateCommentByNotifications 方法,因为这里是管理员登录后台了,有权限更新Comment表了。为了不影响其他模块的,所以这里就会根据Notifications表的发送记录,把Comment表的isNotified更新为true

iouAkira avatar Sep 06 '19 04:09 iouAkira

思路不错

DesertsP avatar Sep 07 '19 08:09 DesertsP

思路不错

@DesertsP 本来想合进去的,感觉好像没什么人有这个需求,我不会Nodejs也是模仿着改的,代码写的也不行,所以提了PR又关了😂,如果有人有这个需求你可以考虑合进去。

iouAkira avatar Sep 09 '19 02:09 iouAkira

@iouAkira 太赞了,我刚好想把它封装成 Docker 镜像,没想到你已经做了,赞

KimYangOfCat avatar Aug 09 '21 00:08 KimYangOfCat