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: 添加内置gocq #1123

Closed
wants to merge 5 commits into from
Closed

feat: 添加内置gocq #1123

wants to merge 5 commits into from

Conversation

Fripine
Copy link
Member

@Fripine Fripine commented Nov 8, 2024

前端部分:sealdice/sealdice-ui#245
解决:
1、sealdice/sealdice-android#36
2、m芯片mac用户无法使用内置协议端

保留代码,先close,寻求更好的解决方案

@Fripine Fripine marked this pull request as draft November 8, 2024 10:06
@Xiangze-Li
Copy link
Member

如果是重用之前的 gocq 还较好。但是从代码来看实际上是新增了第三种内置客户端,这就很不好。

@Fripine
Copy link
Member Author

Fripine commented Nov 8, 2024

如果是重用之前的 gocq 还较好。但是从代码来看实际上是新增了第三种内置客户端,这就很不好。

新增的gocq是lagrange组织基于lagrangego修复的,按照现有的内置客户端的启动流程即可正常使用,通过海豹启动gocq可直接复用platform_adapter_lagrange_helper.go的代码。

对我而言,直接在platform_adapter_lagrange_helper.go的基础上启动gocq,会比去修改旧gocq那部分代码更省开发时间。如果要求对旧gocq进行修改来实现新版gocq的启动,那我需要些时间读懂先前的代码,或由先前gocq部分代码的开发者来实现也行。

先行close

@Fripine Fripine closed this Nov 8, 2024
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