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

[Video] Invert usage of setAudioActive #4924

Merged
merged 1 commit into from
Aug 12, 2024
Merged

Conversation

haileyok
Copy link
Contributor

Typo, should have been the other way around for true/false

Copy link

render bot commented Aug 12, 2024

@haileyok haileyok requested a review from mozzius August 12, 2024 21:53
Copy link
Member

@mozzius mozzius left a comment

Choose a reason for hiding this comment

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

I feel so stupid because I noticed that when you first added it, but assumed you'd meant to do that!!

@haileyok
Copy link
Contributor Author

dont worry i feel just as stupid 😅

@haileyok haileyok merged commit 134fcd3 into main Aug 12, 2024
6 checks passed
@haileyok haileyok deleted the hailey/invert-audio-active branch August 12, 2024 21:58
Copy link

Old size New size Diff
7.09 MB 7.09 MB 0 B (0.00%)

estrattonbailey added a commit that referenced this pull request Aug 15, 2024
* origin/main: (28 commits)
  swap control files (#4936)
  [Embed] Starter pack embed embed (#4935)
  [Video] set audio category to ambient every time a new player is made (#4934)
  Add `/live/` to supported YouTube embed URLs (#4932)
  [Video] Try/catch video play/pause (#4930)
  Don't kick to login screen on network error (#4911)
  Remove .withProxy() calls (#4929)
  [Video] Audio duck off main thread (#4926)
  subclass agent to add setPersistSessionHandler (#4928)
  [Video] Fix crash when switching tabs (#4925)
  [Video] Invert usage of `setAudioActive` (#4924)
  Upgrade API, implement XRPC rework (#4857)
  rm from swift (#4923)
  Fix Android composer cursor bug by removing `setTimeout` from native composer `onChangeText` (#4922)
  Show handle in recent searches and fix truncation (#4917)
  Improve styles (#4916)
  Mark string for localization (#4920)
  Fix `occurred` typo (#4919)
  [Video] Resume background audio whenever muting video audio (#4915)
  [Videos] Add error boundary to native (#4914)
  ...
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