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.

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

feat(android/ios): start/stop recording events for native #15598

Merged

Conversation

Calinteodor
Copy link
Contributor

No description provided.

@Calinteodor Calinteodor force-pushed the native-events-for-recording-control branch 4 times, most recently from 90a2a53 to be62445 Compare February 12, 2025 15:47
@Calinteodor Calinteodor requested a review from saghul February 13, 2025 07:53
ios/sdk/src/ExternalAPI.m Outdated Show resolved Hide resolved
ios/sdk/src/ExternalAPI.m Outdated Show resolved Hide resolved
ios/sdk/src/ExternalAPI.m Outdated Show resolved Hide resolved
react/features/screenshot-capture/functions.tsx Outdated Show resolved Hide resolved
@Calinteodor Calinteodor force-pushed the native-events-for-recording-control branch 5 times, most recently from 6b1d1a8 to f929eef Compare February 13, 2025 12:20
Copy link
Member

@saghul saghul left a comment

Choose a reason for hiding this comment

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

Excellent work! Please tame the linter and drop the ignores before landing.

@Calinteodor Calinteodor force-pushed the native-events-for-recording-control branch 2 times, most recently from 56add84 to b334a0a Compare February 13, 2025 12:54
@Calinteodor Calinteodor force-pushed the native-events-for-recording-control branch 4 times, most recently from 92c3e5b to 7215789 Compare February 13, 2025 14:00
@Calinteodor Calinteodor force-pushed the native-events-for-recording-control branch 6 times, most recently from 998cfa8 to 5c4ec5a Compare February 13, 2025 16:03
@Calinteodor Calinteodor force-pushed the native-events-for-recording-control branch from 5c4ec5a to f623052 Compare February 13, 2025 16:04
@Calinteodor Calinteodor merged commit ef138fb into jitsi:master Feb 13, 2025
10 checks passed
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