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

群内@人文本消息无效 #82

Open
isMeTang opened this issue Dec 9, 2020 · 4 comments
Open

群内@人文本消息无效 #82

isMeTang opened this issue Dec 9, 2020 · 4 comments

Comments

@isMeTang
Copy link

isMeTang commented Dec 9, 2020

debug 跟进发现这个方法的mentionList参数没有用到

override fun messageSendText(conversationId: String, text: String, mentionList: List<String>?): Future<String?> {
        val request = Message.MessageSendTextRequest.newBuilder()
            .setConversationId(conversationId)
            .setText(text)
           //遗漏了这行
            .addAllMentonalIds(mentionList)
            .build()


        return CompletableFuture.supplyAsync {
            val response = grpcClient!!.messageSendText(request)
            val stringValue = response.id
            stringValue.value
        }
    }
@diaozxin007
Copy link
Collaborator

可否提一个 pr 修复这个 bug

isMeTang pushed a commit to isMeTang/java-wechaty that referenced this issue Dec 17, 2020
@EvilBT
Copy link

EvilBT commented Apr 26, 2021

@isMeTang 目前在基于web协议的情况下,添加了这行代码依然@人无效,不会有提示你被@了

@diaozxin007
Copy link
Collaborator

@EvilBT 看了一下应该是 @isMeTang 没有签署 cla 所以代码没有 merge

@EvilBT
Copy link

EvilBT commented Apr 26, 2021

@diaozxin007 我直接在我本地添加了他修改的那行代码,运行的时候照样没有@人效果,不过群里说了,好像web协议本身就不支持@人

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

No branches or pull requests

3 participants