search

itwanger / paicoding

⭐️一款好用又强大的开源社区，基于 Spring Boot、MyBatis-Plus、MySQL、Redis、ElasticSearch、MongoDB、Docker、RabbitMQ 等主流技术栈，附详细教程，包括Java、Spring、MySQL、Redis、微服务&分布式、消息队列等核心知识点。学编程，就上技术派😁。
https://paicoding.com
Apache License 2.0
2.1k stars 398 forks source link

删除冗余接口 #44

Closed qing-wq closed 1 year ago

qing-wq commented 1 year ago
  1. 删除冗余的微信登录接口，并整理写法
  2. 对于CommentWriteServiceImpl.java中删除评论和回复事件，Listener中缺少相应的case
liuyueyi commented 1 year ago
  1. 上面的mr有两个功能改动，一个是删除评论消息，一个是登录的冗余清理 ， 一般这种不同的改动，建议分开提MR
  2. 我记得现在应该没实现删除评论的支持，这个除了消息事件监听之外，还需要考虑支持评论删除，删除某个评论之后，下面的回复是怎样的？前台显示样式是怎样的
  3. 删除登录中的冗余代码时，可以再进一步，将验证码、session、SseEmitter 等缓存结构的设计重构一下，前端登录展示二维码时，可以将两次调用合并为一个
qing-wq commented 1 year ago

好的我明白了