New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Change default value of msg.content to 'None' #496

Merged

rayrayraykk merged 1 commit into alibaba:master from xieyxclack:hotfix_msg

Jan 16, 2023

Collaborator

xieyxclack commented Jan 16, 2023

Change the default value of msg.content from None to 'None' to tackle the issue caused by the fact that grpc cannot support NoneType. (Fix #488 )


          change default value of msg.content to 'None'

68abed5

xieyxclack added the bug label

xieyxclack requested a review from rayrayraykk

January 16, 2023 03:05

rayrayraykk approved these changes

View reviewed changes

Collaborator

rayrayraykk left a comment

LGTM

rayrayraykk merged commit 793f90a into alibaba:master

xieyxclack deleted the hotfix_msg branch

January 16, 2023 05:51

# for free to join this conversation on GitHub. Already have an account? # to comment

Labels

bug