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

website: 小组讨论详情页 #112

Open
2 tasks
Ayase-252 opened this issue Jul 27, 2022 · 7 comments · Fixed by #113
Open
2 tasks

website: 小组讨论详情页 #112

Ayase-252 opened this issue Jul 27, 2022 · 7 comments · Fixed by #113
Milestone

Comments

@Ayase-252
Copy link
Contributor

Ayase-252 commented Jul 27, 2022

设计: https://www.figma.com/file/jW61odhQl5lZzhY4CqvvSF/Web-Page?node-id=1272%3A3740&t=tKoW0IziY9Rn1FBQ-0

TODO:

  • 同步设计稿新改动
  • reactions

reactions

设计稿 https://www.figma.com/file/jW61odhQl5lZzhY4CqvvSF?node-id=1376:5183#414367557
API 见新响应新字段 #540
这里的值跟具体表情之间要map好几次,我不太清除为什么sai老板一开始要这么写...
https://github.com/trim21/bangumi-server-private/blob/478f6d8e298c434a197c13456b160a46f606e9dd/lib/orm/entity/like.ts#L33

@trim21
Copy link
Contributor

trim21 commented Jul 27, 2022

好像所有issue的figma链接都是一样的… b38

@Ayase-252
Copy link
Contributor Author

没看到锚点 b38

@trim21
Copy link
Contributor

trim21 commented Dec 1, 2022

回帖什么的还没做,再打开了

@trim21
Copy link
Contributor

trim21 commented Dec 10, 2022

This was referenced Feb 26, 2023
@trim21 trim21 added this to the v0.0.1 milestone Mar 3, 2023
@trim21 trim21 modified the milestones: 0.0.0-alpha.0, next-next, next Mar 19, 2023
@y-young
Copy link
Contributor

y-young commented May 7, 2023

Reactions 我先认领了,之前做了组件还没做完,有空了就开工

@trim21
Copy link
Contributor

trim21 commented May 7, 2023

Reactions 我先认领了,之前做了组件还没做完,有空了就开工

增删的 API 还没做

@y-young
Copy link
Contributor

y-young commented May 7, 2023

Reactions 我先认领了,之前做了组件还没做完,有空了就开工

增删的 API 还没做

先改 UI 和支持显示吧

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 a pull request may close this issue.

3 participants