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

fix: 修复图片发送问题 #35

Merged
merged 2 commits into from
Jan 17, 2024
Merged

fix: 修复图片发送问题 #35

merged 2 commits into from
Jan 17, 2024

Conversation

leochen-g
Copy link
Collaborator

No description provided.

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jan 14, 2024
@CLAassistant
Copy link

CLAassistant commented Jan 14, 2024

CLA assistant check
All committers have signed the CLA.

@leochen-g leochen-g requested a review from huan January 14, 2024 12:09
src/puppet-oa.ts Outdated Show resolved Hide resolved
@huan
Copy link
Member

huan commented Jan 14, 2024

Please make sure the following requirements have been met for your PR before we can continue reviewing it:

  1. You have signed the CLA by clicking the [CLA | not signed yet] button from the comments.
  2. The PR has passed all the CI tests and turns green. (you can execute all tests locally by running npm test)

@leochen-g
Copy link
Collaborator Author

Please make sure the following requirements have been met for your PR before we can continue reviewing it:

  1. You have signed the CLA by clicking the [CLA | not signed yet] button from the comments.
  2. The PR has passed all the CI tests and turns green. (you can execute all tests locally by running npm test)

因为测试用例使用的是 @huan 的测试号,但是现在测试号是不在线的,所以目前测试案例跑不通 😂

Copy link
Member

@huan huan left a comment

Choose a reason for hiding this comment

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

LGTM

thanks for the improvements!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jan 17, 2024
@huan huan merged commit 4bbd9a7 into wechaty:main Jan 17, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants