Skip to content

企业微信和第三方平台功能增强#2307

Closed
cqyisbug wants to merge 9 commits intobinarywang:developfrom
cqyisbug:develop
Closed

企业微信和第三方平台功能增强#2307
cqyisbug wants to merge 9 commits intobinarywang:developfrom
cqyisbug:develop

Conversation

@cqyisbug
Copy link
Copy Markdown

@cqyisbug cqyisbug commented Sep 8, 2021

  1. 添加企业微信 Spring boot starter,添加基础依赖 spring boot starter
  2. 添加第三方开放平台部分云函数接口
  3. 重构企业微信 WxCpMessage,支持最新企业微信文档的多种消息发送模式
  4. 优化XStream相关工具类

2. 添加第三方开放平台部分云函数接口
3. 重构企业微信 WxCpMessage,支持最新企业微信文档的多种消息发送模式
4. 优化XStream相关工具类
之前用的自己的私有仓库,忘记改回来了
@huayanYu
Copy link
Copy Markdown

huayanYu commented Sep 8, 2021

觉得挺不错,就是一次性提交太多,没法看。

最好能按功能拆分commit, 一个commit实现一个主要功能。
1。 基础模块架子。
2.。 功能1
3 功能2

@cqyisbug
Copy link
Copy Markdown
Author

cqyisbug commented Sep 8, 2021

觉得挺不错,就是一次性提交太多,没法看。

最好能按功能拆分commit, 一个commit实现一个主要功能。
1。 基础模块架子。
2.。 功能1
3 功能2

哎,本来没想着pr,自己用用算了,毕竟改的挺多,有些东西是按照自己想法来的。最近发现合并主仓库代码越来越麻烦,就想着pr一下

Copy link
Copy Markdown

@huayanYu huayanYu left a comment

Choose a reason for hiding this comment

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

觉得挺不错,就是一次性提交太多,没法看。
最好能按功能拆分commit, 一个commit实现一个主要功能。
1。 基础模块架子。
2.。 功能1
3 功能2

哎,本来没想着pr,自己用用算了,毕竟改的挺多,有些东西是按照自己想法来的。最近发现合并主仓库代码越来越麻烦,就想着pr一下

恩,参与开源肯定费事麻烦。 不过就靠大家参与才能更好啊。

public boolean ignoreMatch() {
return false;
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

不太理解为什么要有一个空的实现的抽象类

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

加抽象类是不让使用者迷惑,继承之后只要修改对应的条件函数就行,其他的函数都可以忽略,没有这个抽象类,直接实现接口,实现类里面会有很多的不想干方法

@binarywang
Copy link
Copy Markdown
Owner

非常谢谢你的代码,实在是太多,无法一一查看。
如果可以按功能拆分多次提交,会非常有用。
先关闭此PR。

@binarywang binarywang closed this Sep 10, 2021
Comment thread pom.xml
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.

3 participants