[BUG] Regarding the problem that many SDKs cannot be used in openim-uniapp-polyfill

Ru1ers commented 4 hours ago

OpenIM Server Version

3.8.2

Operating System and CPU Architecture

Windows (AMD)

Deployment Method

Docker Deployment

Bug Description and Steps to Reproduce

使用部分sdk时，控制台报错Cannot read properties of undefined (reading 'apply') 报错位置：

情景：在uniapp中使用openim-uniapp-polyfill 测试平台：web api列表： 1. 会话相关(未选中都是有问题的)

[ ] getAllConversationList
[x] getConversationListSplit
[x] getOneConversation
[ ] getMultipleConversation
[ ] getConversationIDBySessionType
[x] getTotalUnreadMsgCount
[x] markConversationMessageAsRead
[ ] setConversationDraft
[x] setConversation
[ ] hideConversation
[x] changeInputStates
[ ] hideAllConversation(此sdk文档中没有具体说明)
[x] clearConversationAndDeleteAllMsg
[ ] getInputstates
[x] deleteConversationAndDeleteAllMsg

Screenshots Link

目前只测试了会话相关。补充说明：部分sdk可以调用成功，所以证明调用方式是没有问题的。还发现了示例项目中有问题的sdk都没有用到，是否可以理解为这些sdk不支持uniapp，但是文档中uniapp相关确实存在这些sdk。

OpenIM-Robot commented 4 hours ago

Hello! Thank you for filing an issue.

If this is a bug report, please include relevant logs to help us debug the problem.

Join slack 🤖 to connect and communicate with our developers.

Ru1ers commented 4 hours ago

我发现在openim-uniapp-polyfill中有判断，如果是web会指向@openim/client-sdk 而我在@openim/client-sdk的代码中发现，对于会话的sdk声明和文档相比缺少了很多，而且缺少的恰好是我上面提到出问题的sdk。

OpenIM-Robot commented 4 hours ago

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿

I found that there is a judgment in openim-uniapp-polyfill. If it is web, it will point to @openim/client-sdk. And I found in the code of @openim/client-sdk that the SDK declaration and documentation for the session are missing a lot, and what is missing happens to be the SDK that I mentioned above.

Ru1ers commented 3 hours ago

测试在示例项目中使用上面提到的sdk，也会出现问题。 sdk并没有和文档同步。是否补充sdk来和文档保持一致。或者更改文档，去掉不支持的sdk。

openimsdk / open-im-uniapp-demo