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

feat: reactions dropdown & list #582

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

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

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