Skip to content

fix: make consistent use of message composition related props #2695

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

Open
wants to merge 3 commits into
base: rc
Choose a base branch
from

Conversation

MartinCupela
Copy link
Contributor

🎯 Goal

The MessageInputProps should be correctly reflected in TextComposerArea.
Use the TextComposer.insertText method instead of insertText function created by a React hook.

Copy link

codecov bot commented May 2, 2025

Codecov Report

Attention: Patch coverage is 90.47619% with 2 lines in your changes missing coverage. Please review.

Project coverage is 81.80%. Comparing base (a88e145) to head (88c73ae).
Report is 12 commits behind head on rc.

Files with missing lines Patch % Lines
...c/components/TextareaComposer/TextareaComposer.tsx 71.42% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##               rc    #2695      +/-   ##
==========================================
+ Coverage   81.01%   81.80%   +0.78%     
==========================================
  Files         470      463       -7     
  Lines        9925     9182     -743     
  Branches     2413     2093     -320     
==========================================
- Hits         8041     7511     -530     
- Misses       1498     1566      +68     
+ Partials      386      105     -281     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant