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

[gRPC] 补充搜索子区直播信息定义 #1058

Closed
wants to merge 2 commits into from

Conversation

Richasy
Copy link

@Richasy Richasy commented Jul 17, 2024

做了什么

添加了 SearchLiveRoom 的定义,并补充 Item 类型缺失的 live_room 字段。

为什么

目前如果用 SearchByType api 搜索直播相关的内容会无法获取,因为直播间列表会以 SearchLiveRoom 而不是 SearchLiveCard 显示。

@cxw620
Copy link
Contributor

cxw620 commented Jul 18, 2024

#1054 已经包含了这一改动

@Richasy
Copy link
Author

Richasy commented Jul 18, 2024

#1054 已经包含了这一改动

@cxw620

时间上完美错过 😂,我昨天第一时间试了你提交的proto文件列表,测试后dynamic的module内容缺失,所以回退之后自己解包补全了直播搜索的数据结构。

看了下commit历史,我昨天试的版本是 feat: update grpc protos from [email protected] (generated with helper tool),现在已经fix了。

你做了很棒的工作,谢谢!

@Richasy Richasy closed this Jul 18, 2024
@cxw620
Copy link
Contributor

cxw620 commented Jul 18, 2024

#1054 已经包含了这一改动

@cxw620

时间上完美错过 😂,我昨天第一时间试了你提交的proto文件列表,测试后dynamic的module内容缺失,所以回退之后自己解包补全了直播搜索的数据结构。

看了下commit历史,我昨天试的版本是 feat: update grpc protos from [email protected] (generated with helper tool),现在已经fix了。

你做了很棒的工作,谢谢!

因为那个时候我才发现没把 oneof 的 fields 给输出来😄

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 this pull request may close these issues.

2 participants