add feishu plugin to SAA #149
Conversation
|
|
|
you can run |
| <parent> | ||
| <groupId>org.springframework.boot</groupId> | ||
| <artifactId>spring-boot-starter-parent</artifactId> | ||
| <version>3.1.1</version> |
There was a problem hiding this comment.
be consistent with the boot version of spring ai alibaba
btw, you can see:
There was a problem hiding this comment.
ok,I will update this groupid
| <java.version>17</java.version> | ||
| </properties> | ||
|
|
||
| <dependencyManagement> |
There was a problem hiding this comment.
According to the transitive relationship of mvn, after introducing the above parent, there is no need to introduce it again here.
| </dependency> | ||
| <dependency> | ||
| <groupId>com.fasterxml.jackson.core</groupId> | ||
| <artifactId>jackson-annotations</artifactId> |
There was a problem hiding this comment.
We shouldn't need this dependency. maybe can delete it
| <artifactId>jsoup</artifactId> | ||
| <version>1.18.1</version> | ||
| </dependency> | ||
| <dependency> |
There was a problem hiding this comment.
ok， but I saw that this dependency was used in the other plugins in the project
| @@ -0,0 +1,101 @@ | |||
| package com.alibaba.cloud.ai.plugin.feishu; | |||
|
|
|||
| import com.google.gson.JsonParser; | |||
There was a problem hiding this comment.
I see that you have used jackson. The global situation should be unified. There is no need for gson here. What do you think?
| @Bean | ||
| @ConditionalOnMissingBean | ||
| @Description("Build FeiShu Client in Spring AI Alibaba")// description | ||
| @ConditionalOnProperty( |
There was a problem hiding this comment.
As a plugin, I don’t think we need this configuration. cc @chickenlj
There was a problem hiding this comment.
In reader of Feishu,users need to configure their own appid and appSecret
| * @author <a href="mailto:[email protected]">wudihaoke214</a> | ||
| */ | ||
| @SpringBootTest | ||
| class SpringAiAlibabaStarterPluginFeiShuApplicationTests { |
There was a problem hiding this comment.
If there is no test class, you can delete it
add feishu plugin to SAA