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

[go][iOS] Handle initialUrl passed in the argv #33182

Merged
merged 1 commit into from
Nov 25, 2024

Conversation

lukmccall
Copy link
Contributor

Why

Handles the initialUrl argument when passed as a process argument.

In the Radeon IDE, the application is currently opened via a deep link, which presents a significant drawback: there is no way to access the process's stdout. The IDE team would like to launch the application using xcrun simctl launch to access process logs and capture native errors. This pull request adds support for an initialUrl parameter passed in the argv.

How

Parsed the argv when calculating the initialUrl.

Test Plan

  • launch app from icon ✅
  • launch app using xcrun simctl launch
     xcrun simctl launch <device_id> host.exp.Exponent --initialUrl "exp://192.168.0.235:8081"
    
  • launch app using deep link ✅
     xcrun simctl openurl booted exp://192.168.0.235:8081
    

@lukmccall lukmccall requested review from Kudo and tsapeta November 22, 2024 13:38
@expo-bot expo-bot added the bot: passed checks ExpoBot has nothing to complain about label Nov 22, 2024
@lukmccall lukmccall merged commit bba56a4 into main Nov 25, 2024
14 checks passed
@lukmccall lukmccall deleted the @lukmccall/go/add-inital-url-arg branch November 25, 2024 11:12
lukmccall added a commit that referenced this pull request Nov 26, 2024
# Why

Handles the `initialUrl` argument when passed as a process argument.

In the Radeon IDE, the application is currently opened via a deep link, which presents a significant drawback: there is no way to access the process's stdout. The IDE team would like to launch the application using `xcrun simctl launch` to access process logs and capture native errors. This pull request adds support for an `initialUrl` parameter passed in the `argv`.

# How

Parsed the `argv` when calculating the `initialUrl`. 

# Test Plan

- launch app from icon ✅ 
- launch app using `xcrun simctl launch` ✅
	```
	xcrun simctl launch <device_id> host.exp.Exponent --initialUrl "exp://192.168.0.235:8081"
	```
- launch app using deep link ✅ 
	```
	xcrun simctl openurl booted exp://192.168.0.235:8081
	```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot: passed checks ExpoBot has nothing to complain about
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants