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

支持私有安装版企业微信 #1063

Closed
wants to merge 1 commit into from

Conversation

crazycode
Copy link

项目中有接触到客户部署了私有化部署的企业微信,而WxJava中服务地址 https://qyapi.weixin.qq.com 是固定写死的,导致无法使用WxJava。

产品说明见:https://work.weixin.qq.com/server

为此,对项目做了修改,供您参考。

使用方法:

通过 wxCpConfigStorage.setBaseApiUrl("http://local_server:port"); 设置私有化部署的企业微信服务地址.

默认值是 https://qyapi.weixin.qq.com , 如果使用默认值,则不需要调用 setBaseApiUrl(baseUrl).

通过 wxCpConfigStorage.setBaseApiUrl("http://local_server:port"); 设置私有化部署的企业微信服务地址.

默认值是 https://qyapi.weixin.qq.com , 如果使用默认值,则不需要调用 setBaseApiUrl(baseUrl).
@crazycode
Copy link
Author

自动测试不通过?我哪里写错了吗?

@binarywang
Copy link
Owner

有部分冲突,因为这部分我正好在优化,你可以重新获取下最新代码,然后再处理下

@binarywang binarywang closed this Jun 2, 2019
@crazycode
Copy link
Author

这个部分不再修改了吧?冲突是有点多。

@binarywang
Copy link
Owner

本来今天是想做比较大的优化的,你先提交你的现有代码吧,之后我再做统一优化

@crazycode
Copy link
Author

好的,稍等,我在进行中,谢谢。

@crazycode
Copy link
Author

`
public interface WxCpConfigStorage {

/**

  • 设置企业微信服务器 baseUrl.
  • 默认值是 https://qyapi.weixin.qq.com , 如果使用默认值,则不需要调用 setBaseApiUrl
  • @param baseUrl 企业微信服务器 Url
    */
    void setBaseApiUrl(String baseUrl);

/**

  • 读取企业微信 API Url.
  • 支持私有化企业微信服务器.
    */
    String getApiUrl(String path);
    `
    这段写法没什么问题吧?

@binarywang
Copy link
Owner

看着没什么问题,命名也很到位

@crazycode
Copy link
Author

还有一会才能修改完,您这次重构有点大啊

@crazycode crazycode deleted the develop branch June 2, 2019 05:39
@binarywang
Copy link
Owner

没事,不急,等你弄完我晚上再发布个版本,我这边暂时先不急着做进一步的重构

@crazycode
Copy link
Author

crazycode commented Jun 2, 2019

好了,已经 pull request #1065

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