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

Restore /launch-js-devtools endpoint (revert #2065) #2083

Merged

Conversation

huntie
Copy link
Collaborator

@huntie huntie commented Sep 21, 2023

Summary

Test plan

  • iOS JSC build
  • Enable remote debugging via NativeDevSettings.setIsDebuggingRemotely(true)

✅ Auto-launches /debugger-ui, connects external JS runtime

image

Copy link
Member

@thymikee thymikee left a comment

Choose a reason for hiding this comment

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

Ok 😅

@thymikee thymikee merged commit 9733b6f into react-native-community:main Sep 21, 2023
6 checks passed
@huntie huntie deleted the deprecate-remote-debugging branch September 21, 2023 12:30
huntie added a commit to huntie/react-native that referenced this pull request Sep 21, 2023
Summary:
Some tidying up around remote JS debugging while testing. See also react-native-community/cli#2083.

Changelog: [Internal]

Differential Revision: D49499068

fbshipit-source-id: 2e3c0a0e059333b95fca8ffbc2537ec5f01384c1
huntie added a commit to huntie/react-native that referenced this pull request Sep 21, 2023
facebook#39582)

Summary:
Pull Request resolved: facebook#39582

Some tidying up around remote JS debugging while testing. See also react-native-community/cli#2083.

## Changelog:
[iOS][Breaking] - Remove `openURL` method from `RCTInspectorDevServerHelper`

Differential Revision: D49499068

fbshipit-source-id: 8ab000439133b7131c6b7310f87ff6229a57118d
huntie added a commit to huntie/react-native that referenced this pull request Sep 21, 2023
facebook#39582)

Summary:
Pull Request resolved: facebook#39582

Some tidying up around remote JS debugging while testing. See also react-native-community/cli#2083.

## Changelog:
[iOS][Breaking] - Remove `openURL` method from `RCTInspectorDevServerHelper`

Reviewed By: cipolleschi

Differential Revision: D49499068

fbshipit-source-id: b926bec49e2c67d2bc6662d32178cbb5baed2121
huntie added a commit to huntie/react-native that referenced this pull request Sep 21, 2023
facebook#39582)

Summary:
Pull Request resolved: facebook#39582

Some tidying up around remote JS debugging while testing. See also react-native-community/cli#2083.

## Changelog:
[iOS][Breaking] - Remove `openURL` method from `RCTInspectorDevServerHelper`

Reviewed By: cipolleschi

Differential Revision: D49499068

fbshipit-source-id: 662ca6353837a8423c67dd58003c376b0ae21ee6
huntie added a commit to huntie/react-native that referenced this pull request Sep 21, 2023
facebook#39582)

Summary:
Pull Request resolved: facebook#39582

Some tidying up around remote JS debugging while testing. See also react-native-community/cli#2083.

## Changelog:
[iOS][Breaking] - Remove `openURL` method from `RCTInspectorDevServerHelper`

Reviewed By: cipolleschi

Differential Revision: D49499068

fbshipit-source-id: cb700125c5c4818f51241f78f98e566fe355ee61
huntie added a commit to huntie/react-native that referenced this pull request Sep 22, 2023
facebook#39582)

Summary:

Some tidying up around remote JS debugging while testing. See also react-native-community/cli#2083.

## Changelog:
[iOS][Breaking] - Remove `openURL` method from `RCTInspectorDevServerHelper`

Reviewed By: cipolleschi

Differential Revision: D49499068
huntie added a commit to huntie/react-native that referenced this pull request Sep 22, 2023
facebook#39582)

Summary:

Some tidying up around remote JS debugging while testing. See also react-native-community/cli#2083.

## Changelog:
[iOS][Breaking] - Remove `openURL` method from `RCTInspectorDevServerHelper`

Reviewed By: cipolleschi

Differential Revision: D49499068
facebook-github-bot pushed a commit to facebook/react-native that referenced this pull request Sep 22, 2023
#39582)

Summary:
Pull Request resolved: #39582

Some tidying up around remote JS debugging while testing. See also react-native-community/cli#2083.

## Changelog:
[iOS][Breaking] - Remove `openURL` method from `RCTInspectorDevServerHelper`

Reviewed By: cipolleschi

Differential Revision: D49499068

fbshipit-source-id: 843ef6cf1af658756466854fc1a0b3edd91c1f37
ShevO27 pushed a commit to ShevO27/react-native that referenced this pull request Sep 26, 2023
facebook#39582)

Summary:
Pull Request resolved: facebook#39582

Some tidying up around remote JS debugging while testing. See also react-native-community/cli#2083.

## Changelog:
[iOS][Breaking] - Remove `openURL` method from `RCTInspectorDevServerHelper`

Reviewed By: cipolleschi

Differential Revision: D49499068

fbshipit-source-id: 843ef6cf1af658756466854fc1a0b3edd91c1f37
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