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

修复群中一旦有@就导致消息接收慢的问题 #208

Merged
merged 5 commits into from
Dec 11, 2022

Conversation

leochen-g
Copy link
Contributor

@leochen-g leochen-g requested a review from a team as a code owner December 10, 2022 07:09
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.

Could you please bump the minor version to a new odd number so that we can make sure this change will not affect the old version users when we are testing?

thank you very much for this great improvement!

@leochen-g
Copy link
Contributor Author

Could you please bump the minor version to a new odd number so that we can make sure this change will not affect the old version users when we are testing?

thank you very much for this great improvement!

ok, has changed

@huan
Copy link
Member

huan commented Dec 11, 2022

Could you please bump the minor version to a new odd number so that we can make sure this change will not affect the old version users when we are testing?
thank you very much for this great improvement!

ok, has changed

Please change 1.20.5 to 1.21.0 because the middle version number is the minor number.

You can use the command to bump it:

npm version minor

The last number is the patch number.

@leochen-g
Copy link
Contributor Author

Could you please bump the minor version to a new odd number so that we can make sure this change will not affect the old version users when we are testing?
thank you very much for this great improvement!

ok, has changed

Please change 1.20.5 to 1.21.0 because the middle version number is the minor number.

You can use the command to bump it:

npm version minor

The last number is the patch number.

Ok, Thank you for your reminder.

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

Thank you so much for fixing this issue!

@huan huan merged commit 53a8159 into wechaty:main Dec 11, 2022
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