Skip to content

Call getCurrent on mount to drain initial deep link URL#716

Closed
DeepanshuMishraa wants to merge 2 commits into
athasdev:masterfrom
DeepanshuMishraa:fix-cli
Closed

Call getCurrent on mount to drain initial deep link URL#716
DeepanshuMishraa wants to merge 2 commits into
athasdev:masterfrom
DeepanshuMishraa:fix-cli

Conversation

@DeepanshuMishraa

Copy link
Copy Markdown
Contributor

onOpenUrl only fires for URLs received while the app is already running. When Athas is cold-launched via a CLI command, the URL that launched the app is only available through getCurrent. Without this call, the initial deep link URL is silently dropped and the user sees an empty editor.

Fixes athas open ., athas ., athas terminal, athas web, athas remote on cold start.

onOpenUrl only fires for URLs received while the app is already running.
When Athas is cold-launched via a CLI command, the URL that launched the
app is only available through getCurrent. Without this call, the initial
deep link URL is silently dropped and the user sees an empty editor.

Fixes athas open ., athas ., athas terminal, athas web, athas remote
on cold start.
@mehmetozguldev

Copy link
Copy Markdown
Member

This should be recreated against main since we changed the branch name from master to main :(

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