refactor: replace ReactElement type annotations with ReactNode in functions#40817
refactor: replace ReactElement type annotations with ReactNode in functions#40817tassoevan wants to merge 15 commits into
Conversation
… from AppAvatar, MessageAvatar, and RoomAvatar components
…ns across components
…ions across components
…nnotations across registration components
…ations in components - Updated multiple components to remove explicit return type annotations of ReactElement, allowing TypeScript to infer the return type. - Adjusted BlockProps to use ReactElement<any> for better type compatibility. - Cleaned up imports by removing unused ReactElement imports across various files.
…onal components - Removed explicit return type annotations of ReactElement from multiple functional components across the codebase. - Updated components in Markup, PreviewMarkup, HeadingBlock, OrderedListBlock, ParagraphBlock, QuoteBlock, SpoilerBlock, TaskListBlock, UnorderedListBlock, CodeBlock, CodeElement, PreviewCodeBlock, PreviewCodeElement, ColorElement, PreviewColorElement, BoldSpan, ImageElement, InlineElements, ItalicSpan, LinkSpan, PlainSpan, PreviewInlineElements, SpoilerSpan, StrikeSpan, BigEmojiBlock, BigEmojiElement, Emoji, EmojiElement, EmojiRenderer, PreviewBigEmojiBlock, PreviewEmojiElement, KatexBlock, KatexElement, KatexErrorBoundary, PreviewKatexBlock, PreviewKatexElement, ChannelMentionElement, PreviewChannelMentionElement, PreviewUserMentionElement, UserMentionElement.
…s from components
…tations from components
|
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Looks like this PR is not ready to merge, because of the following issues:
Please fix the issues and try again If you have any trouble, please check the PR guidelines |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## develop #40817 +/- ##
========================================
Coverage 69.90% 69.90%
========================================
Files 3337 3338 +1
Lines 123489 123496 +7
Branches 22013 22007 -6
========================================
+ Hits 86322 86332 +10
- Misses 33813 33824 +11
+ Partials 3354 3340 -14
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Proposed changes (including videos or screenshots)
As a first step towards upgrading to React 19, it handles types from
@types/reactlooking forward the next major.Issue(s)
Task: ARCH-2170
Steps to test or reproduce
Further comments
No runtime change is expected from it.