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.

Sign up for GitHub

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

Jump to bottom

MessageBar calling 'onSendMessage' twice and getting a console error #483

Closed
jeff-phillips-18 opened this issue Mar 16, 2025 · 0 comments · Fixed by #484
Closed

MessageBar calling 'onSendMessage' twice and getting a console error #483

jeff-phillips-18 opened this issue Mar 16, 2025 · 0 comments · Fixed by #484
Assignees
@jeff-phillips-18
Labels
PF6 Applies to only the PF6 version
Milestone
2025.Q1

Comments

@jeff-phillips-18
Copy link
Member

If my onSendMessage function is a react callback, it ends up being called twice when the user adds a message. Also seeing:

Cannot update a component (`ChatModelEval`) while rendering a different component (`MessageBar`). To locate the bad setState() call inside `MessageBar`, follow the stack trace as described in https://react.dev/link/setstate-in-render

in the console window.
@jeff-phillips-18 jeff-phillips-18 self-assigned this Mar 16, 2025
@github-project-automation github-project-automation bot moved this to Needs triage in PatternFly Issues Mar 16, 2025
@jeff-phillips-18 jeff-phillips-18 mentioned this issue Mar 16, 2025
Merged
@rebeccaalpert rebeccaalpert added this to the 2025.Q1 milestone Mar 17, 2025
@rebeccaalpert rebeccaalpert moved this from Needs triage to PR Review in PatternFly Issues Mar 17, 2025
@rebeccaalpert rebeccaalpert added the PF6 Applies to only the PF6 version label Mar 17, 2025
@github-project-automation github-project-automation bot moved this from PR Review to Done in PatternFly Issues Mar 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jeff-phillips-18 jeff-phillips-18

Labels
PF6 Applies to only the PF6 version
Projects
PatternFly Issues
Archived in project
Milestone
2025.Q1
Development

Successfully merging a pull request may close this issue.

fix(MessageBar): Call onSendMessage only once for new messages jeff-phillips-18/chatbot
2 participants
@rebeccaalpert @jeff-phillips-18