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

评论支持joke和emotion类型,增加评论置顶 #20

Closed
iRuxu opened this issue Dec 12, 2021 · 7 comments
Closed

评论支持joke和emotion类型,增加评论置顶 #20

iRuxu opened this issue Dec 12, 2021 · 7 comments
Assignees
Labels
bug Something isn't working 完成待测试

Comments

@iRuxu
Copy link
Member

iRuxu commented Dec 12, 2021

评论支持joke和emotion类型

原先joke和emotion是在wp_posts表中的,要更换。
已独立在主库的 pvx_emotions和pvx_jokes表中。

【主ISSUE】#9
更新如下:
image

新增支持2个对应的评论类型

@iRuxu
Copy link
Member Author

iRuxu commented Dec 12, 2021

评论新增置顶功能

1.评论条目中增加置顶按钮

image

  • 仅原作者可见可用 User.getInfo().uid == 当前作品的作者
  • 置顶图标:el-icon-top
  • 子楼层回复不支持设置置顶

2.工具栏改版

image

3.评论列表

  • 当评论显示时,默认优先显示置顶的评论。
  • 被置顶的条目,在作者名字后增加一个置顶的标记,如下图
    image

@iRuxu
Copy link
Member Author

iRuxu commented Dec 12, 2021

交付时间:期望在周三/四前完成

改完后,最后我来改版本号上线(上线前我可能还会微调一下样式)

@flyflyknife
Copy link
Contributor

评论支持joke和emotion类型

原先joke和emotion是在wp_posts表中的,要更换。 已独立在主库的 pvx_emotions和pvx_jokes表中。

【主ISSUE】#9 更新如下: image

新增支持2个对应的评论类型

完成待上线

flyflyknife pushed a commit that referenced this issue Dec 15, 2021
@iRuxu
Copy link
Member Author

iRuxu commented Dec 16, 2021

置顶权限问题。

image
http://localhost:8081/api/comment/post/article/100/i-am-author
我是管理员,接口返回中有权限

但是实际操作时,如下:
image

  • 已完成

iRuxu added a commit that referenced this issue Dec 16, 2021
@iRuxu iRuxu added bug Something isn't working and removed 完成待测试 进行中 完成待上线 labels Dec 17, 2021
@flyflyknife
Copy link
Contributor

置顶权限问题。

image http://localhost:8081/api/comment/post/article/100/i-am-author 我是管理员,接口返回中有权限

但是实际操作时,如下: image

  • 已完成

这里所以显示bug,你不是作者不应该展示这个

image

@flyflyknife
Copy link
Contributor

已更新 需要重新上线前端

@iRuxu
Copy link
Member Author

iRuxu commented Dec 19, 2021

置顶的仍然有错误。
我现在是19382的作者
image
这里返回的却是100

@iRuxu iRuxu closed this as completed Dec 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working 完成待测试
Projects
None yet
Development

No branches or pull requests

2 participants