Skip to content

fix: After an error on sendMessage, all subsequent calls fail with the same error - #551

Closed
josejefferson wants to merge 3 commits into
googleapis:mainfrom
josejefferson:send-message-fix
Closed

fix: After an error on sendMessage, all subsequent calls fail with the same error#551
josejefferson wants to merge 3 commits into
googleapis:mainfrom
josejefferson:send-message-fix

Conversation

@josejefferson

Copy link
Copy Markdown
Contributor

Fix issue #550 and added test case

@hkt74 hkt74 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for making the fix!

Comment thread test/unit/chats_test.ts Outdated
copybara-service Bot pushed a commit that referenced this pull request May 20, 2025
--
24fc179 by Jefferson Dantas <jeffersonn.jjdaj@gmail.com>:

fix: After an error on sendMessage, all subsequent calls fail with the same error

--
ed227e4 by Jefferson Dantas <jeffersonn.jjdaj@gmail.com>:

fix: Fixed sendMessage subsequent calls test to follow the arrange-act-assert pattern
COPYBARA_INTEGRATE_REVIEW=#551 from josejefferson:send-message-fix 3abc1e3
PiperOrigin-RevId: 761107064
@hkt74

hkt74 commented May 20, 2025

Copy link
Copy Markdown
Collaborator

the change was committed in 778abcc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants