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

Remove .withProxy() calls #4929

Merged
merged 1 commit into from
Aug 13, 2024
Merged

Remove .withProxy() calls #4929

merged 1 commit into from
Aug 13, 2024

Conversation

gaearon
Copy link
Collaborator

@gaearon gaearon commented Aug 13, 2024

We accidentally broke them by subclassing the agent. Could remove the subclass but also we might as well remove this gate (it had no meaningful impact on session events so must've been equivalent).

@gaearon gaearon requested a review from haileyok August 13, 2024 04:25
Copy link

render bot commented Aug 13, 2024

Copy link

Old size New size Diff
7.09 MB 7.09 MB -601 B (-0.01%)

@haileyok haileyok merged commit 7e11b86 into main Aug 13, 2024
6 checks passed
haileyok added a commit that referenced this pull request Aug 14, 2024
(cherry picked from commit 7e11b86)
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)
  ...
estrattonbailey added a commit that referenced this pull request Aug 21, 2024
* origin/main: (50 commits)
  Add `list hidden` screen (#4958)
  Expose more methods, support disabled items (#4954)
  Expose more props from button (#4953)
  Fix orphaned feed slices, handle blocks (#4944)
  Tweak `expo-modules-core` hack patch (#4955)
  [Experiment] Always show bottom bar (#4946)
  Revert "[Video] Download videos" (#4945)
  Move global "Sign out" out of the current account row (#4941)
  Hack patch for testing OTA update crash behavior (#4942)
  [Video] Download videos (#4886)
  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)
  ...
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