Skip to content

🎨 #3624 【企业微信】企微微盘API接口不需要传userId #3625

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

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

benben4466
Copy link

企业微信在2022年12月01号修改了企业微信微盘的API接口,取消接口中userId字段。

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes the now-unused userId parameter from WeDrive API methods to align with the enterprise WeChat change in December 2022.

  • Method signatures in the service interface no longer include userId
  • Implementation no longer sends userid in JSON payloads
  • Test calls updated to match the new signatures

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
weixin-java-cp/src/main/java/me/chanjar/weixin/cp/api/WxCpOaWeDriveService.java Removed userId from method signatures and Javadoc
weixin-java-cp/src/main/java/me/chanjar/weixin/cp/api/impl/WxCpOaWeDriveServiceImpl.java Dropped userid JSON properties and updated signatures
weixin-java-cp/src/test/java/me/chanjar/weixin/cp/api/WxCpOaWeDriveServiceTest.java Updated test invocations to call methods without userId
Comments suppressed due to low confidence (1)

weixin-java-cp/src/main/java/me/chanjar/weixin/cp/api/WxCpOaWeDriveService.java:55

  • Removing these methods entirely can break existing clients. Consider deprecating the old overloads (with userId) and providing the new signatures in parallel to give consumers a migration path.
  WxCpBaseResp spaceDismiss(@NonNull String spaceId) throws WxErrorException;

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.

企微微盘的API更新,所有的接口现在全部都不需要传userId
1 participant