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

重构 openWork 以应对base Client中override access_token 问题 #464

Merged
merged 9 commits into from
Apr 6, 2024

Conversation

bububa
Copy link
Contributor

@bububa bububa commented Apr 4, 2024

另外建议func NewBaseClient(app *ApplicationInterface, token *AccessToken) (*BaseClient, error) 中的 token 参数,带入token 可能会导致不可预知错误 。

Matrix-X and others added 9 commits March 11, 2024 11:10
修正参数类型:公众号发布文章API接口返回的PublishId参数实际为数字型
修正参数类型:公众号发布文章API接口返回的PublishId参数实际为数字型
更新发布文章API接口PublishId参数类型
fix(openplatform): accessToken := (*client.App).GetAccessToken()


1. OpenWorkProvider Application 对应仅使用provider_access_token的API, 如openWork/license内相关API
2. OpenWork Application 紧涉及suite_access_token的API
3. OpenWork.ProviderClient(corpID string, permanentCode string, externalConfig *work.UserConfig) *work.Work 返回服务商待开发应该对应使用work.Work
4. OpenWork.ThirdpartyClient(corpID string, permanentCode string, externalConfig *work.UserConfig) *work.Work 返回第三方开发应用对应使用的work.Work, work的AccessToken替换为openWork/corp.AccessToken, 给token会自动刷新suite_access_token


1. OpenWorkProvider Application 对应仅使用provider_access_token的API, 如openWork/license内相关API
2. OpenWork Application 紧涉及suite_access_token的API
3. OpenWork.ProviderClient(corpID string, permanentCode string, externalConfig *work.UserConfig) *work.Work 返回服务商待开发应该对应使用work.Work
4. OpenWork.ThirdpartyClient(corpID string, permanentCode string, externalConfig *work.UserConfig) *work.Work 返回第三方开发应用对应使用的work.Work, work的AccessToken替换为openWork/corp.AccessToken, 给token会自动刷新suite_access_token
@Matrix-X Matrix-X merged commit 1839fa7 into ArtisanCloud:develop Apr 6, 2024
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