Skip to content

feat: reactions dropdown & list#582

Draft
y-young wants to merge 5 commits intobangumi:masterfrom
y-young:feat/reactions
Draft

feat: reactions dropdown & list#582
y-young wants to merge 5 commits intobangumi:masterfrom
y-young:feat/reactions

Conversation

@y-young
Copy link
Contributor

@y-young y-young commented May 16, 2023

Tracking: #112

  • Reactions 抽取为单独类型
  • SubReply 中 reactions 字段
  • 已选中的 Reaction 样式设计稿
image

@github-actions
Copy link
Contributor

github-actions bot commented May 16, 2023

Preview Deployment

Build URL time
Sites netlify preview 2023-04-01 04:04:16Z
Storybook netlify preview 2023-04-01 04:03:12Z

@trim21
Copy link
Contributor

trim21 commented May 19, 2023

子回复的reactions修复了

@mergify
Copy link
Contributor

mergify bot commented May 19, 2023

@y-young this pull request is now in conflict 😩

@trim21
Copy link
Contributor

trim21 commented May 19, 2023

client文件一直都是自动生成的,为什么会有冲突…

@y-young
Copy link
Contributor Author

y-young commented May 19, 2023

client文件一直都是自动生成的,为什么会有冲突…

因为我之前手动改了(

@y-young
Copy link
Contributor Author

y-young commented May 19, 2023

表情的对应关系想个办法自动同步过来?现在是手动复制的

@trim21
Copy link
Contributor

trim21 commented May 19, 2023

openapi是从响应/请求的校验结构体生成的,如果openapi里没有,那响应里面也不会有…

@y-young
Copy link
Contributor Author

y-young commented May 19, 2023

openapi是从响应/请求的校验结构体生成的,如果openapi里没有,那响应里面也不会有…

我是说 这个 能不能用类似 wiki-syntax-spec 的 submodule 放进来

@trim21
Copy link
Contributor

trim21 commented May 19, 2023

openapi是从响应/请求的校验结构体生成的,如果openapi里没有,那响应里面也不会有…

我是说 这个 能不能用类似 wiki-syntax-spec 的 submodule 放进来

我是说手动修改 client.ts (

这里我想个办法,或者直接在服务端map一次,客户端不需要map了也行

@trim21
Copy link
Contributor

trim21 commented May 19, 2023

这里是sai老板在php里面map了一次,又看了一下可能没什么必要。

直接用 bgm 对应的数字应该就行

@trim21
Copy link
Contributor

trim21 commented May 19, 2023

openapi是从响应/请求的校验结构体生成的,如果openapi里没有,那响应里面也不会有…

我是说 这个 能不能用类似 wiki-syntax-spec 的 submodule 放进来

这个map是从reactions value对应到图片path,比如 reactions value 54 对应 bgm38 对应 https://bgm.tv/img/smiles/tv/15.gif15

这里应该是不需要存这个map的,在bbcode那边已经有一份了,比如把 bgm38 渲染成15.gif

@trim21
Copy link
Contributor

trim21 commented May 19, 2023

可能在这里

if (id >= 40 && id < 140) {

@trim21
Copy link
Contributor

trim21 commented May 19, 2023

实际需要维护的应该只有一个 Array 用来保存允许的值,暂时先硬编码吧,正好还避免了排序问题。

@y-young
Copy link
Contributor Author

y-young commented May 20, 2023

这个map是从reactions value对应到图片path,比如 reactions value 54 对应 bgm38 对应 https://bgm.tv/img/smiles/tv/15.gif15

这里应该是不需要存这个map的,在bbcode那边已经有一份了,比如把 bgm38 渲染成15.gif

但是还需要一个 map 把 54 映射到 bgm38 吧

@trim21
Copy link
Contributor

trim21 commented May 20, 2023

哦哦,你说的对,我理解错了。

这个map感觉前端存一份就够了,后端应该是只存最终允许的值就行

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants