Skip to content

refactor(large-video): migrate LargeVideo.native.tsx to function component#17027

Open
nishant25kr wants to merge 1 commit intojitsi:masterfrom
nishant25kr:refactor/large-video-hooks
Open

refactor(large-video): migrate LargeVideo.native.tsx to function component#17027
nishant25kr wants to merge 1 commit intojitsi:masterfrom
nishant25kr:refactor/large-video-hooks

Conversation

@nishant25kr
Copy link
Contributor

What this PR does

  • Converts LargeVideo.native.tsx from a class component to a function component.
  • Extracts track streaming status logic into a dedicated hook (useLocalTrackStreamingStatus).
  • Preserves existing streaming-status behavior and lifecycle semantics.

Why

  • Aligns the component with modern React hooks patterns.
  • Addresses existing TODOs in the file regarding hook-based implementation.
  • Improves maintainability without changing feature ownership or behavior scope.

Notes on testing

  • Local E2E tests related to audio/video moderation depend on broader Redux updates
    not owned by LargeVideo. This refactor intentionally keeps the hook scoped only
    to track streaming status, as per the original TODO and component responsibility.

@nishant25kr
Copy link
Contributor Author

Fixes #16707

@jitsi-jenkins
Copy link

Hi, thanks for your contribution!
If you haven't already done so, could you please make sure you sign our CLA (https://jitsi.org/icla for individuals and https://jitsi.org/ccla for corporations)? We would unfortunately be unable to merge your patch unless we have that piece :(.

@nishant25kr
Copy link
Contributor Author

Thanks for the heads-up!
I’ve already signed the Individual CLA just now. Please let me know if anything else is needed from my side.

@nishant25kr
Copy link
Contributor Author

Hi! Just a gentle ping in case this got buried.
Happy to address any feedback when you have time — thanks!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants