Skip to content

[patch] add back roundtripview#2

Merged
nsingh-branch merged 2 commits intomainfrom
roundtripview
Dec 2, 2024
Merged

[patch] add back roundtripview#2
nsingh-branch merged 2 commits intomainfrom
roundtripview

Conversation

@bredmond5
Copy link
Copy Markdown
Contributor

This got removed while i was rebasing unfortunately

This got removed while i was rebasing unfortunately
@nsingh-branch nsingh-branch merged commit 1ed08ea into main Dec 2, 2024
@bredmond5 bredmond5 deleted the roundtripview branch December 2, 2024 23:54
wpinho-branch added a commit that referenced this pull request Jan 20, 2026
Changes:
- OLD API now uses Branch.application(_:open:options:) for URL schemes
  (the actual AppDelegate method that would cause double-open)
- OLD API now uses Branch.continue(userActivity) for Universal Links
- Clear step-by-step logging showing:
  - Step 1: initSession(launchOptions: nil) → OPEN #1
  - Step 2: application(_:open:) → OPEN #2 (bug)
- NEW API uses BranchScene.initSession(with:) which handles
  everything in a single OPEN request
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