-
Notifications
You must be signed in to change notification settings - Fork 43
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
base: master
Are you sure you want to change the base?
Conversation
Preview Deployment
|
子回复的reactions修复了 |
@y-young this pull request is now in conflict 😩 |
client文件一直都是自动生成的,为什么会有冲突… |
因为我之前手动改了( |
表情的对应关系想个办法自动同步过来?现在是手动复制的 |
openapi是从响应/请求的校验结构体生成的,如果openapi里没有,那响应里面也不会有… |
我是说 这个 能不能用类似 wiki-syntax-spec 的 submodule 放进来 |
我是说手动修改 client.ts ( 这里我想个办法,或者直接在服务端map一次,客户端不需要map了也行 |
这里是sai老板在php里面map了一次,又看了一下可能没什么必要。 直接用 bgm 对应的数字应该就行 |
这个map是从reactions value对应到图片path,比如 reactions value 54 对应 这里应该是不需要存这个map的,在bbcode那边已经有一份了,比如把 |
可能在这里 frontend/packages/utils/bbcode/convert.ts Line 137 in b530c8e
|
实际需要维护的应该只有一个 Array 用来保存允许的值,暂时先硬编码吧,正好还避免了排序问题。 |
但是还需要一个 map 把 54 映射到 bgm38 吧 |
哦哦,你说的对,我理解错了。 这个map感觉前端存一份就够了,后端应该是只存最终允许的值就行 |
Tracking: #112