Skip to content

Fix a regerssion with univeral links#25302

Open
kean wants to merge 1 commit intorelease/26.7from
fix/stats-universal-link
Open

Fix a regerssion with univeral links#25302
kean wants to merge 1 commit intorelease/26.7from
fix/stats-universal-link

Conversation

@kean
Copy link
Contributor

@kean kean commented Feb 25, 2026

Fixes https://linear.app/a8c/issue/CMM-1286/jetpack-ios-tapping-stats-widget-opens-my-website-instead-of

RCA

When the universal link handling reaches the following code to invoke the deep link, the sections are empty. It appears to be a regression introduced during the Swift rewrite. The sections configuration must be populated in viewDidLoad and not it does it only on viewWillAppear.

Screenshot 2026-02-25 at 11 23 10 AM

The fix is more of a workaround. It's not ideal that universal links depend on sections or rely on BlogDetailsViewController in the first place. The app needs to adopt something more like the Reader navigation system with a root coordinator handling every single possible route.

Recording

Screen.Recording.2026-02-25.at.11.27.19.AM.mov

@kean kean added this to the 26.7 ❄️ milestone Feb 25, 2026
@kean kean requested a review from crazytonyli February 25, 2026 16:31
@dangermattic
Copy link
Collaborator

1 Warning
⚠️ This PR is assigned to the milestone 26.7 ❄️. The due date for this milestone has already passed.
Please assign it to a milestone with a later deadline or check whether the release for this milestone has already been finished.

Generated by 🚫 Danger

@wpmobilebot
Copy link
Contributor

App Icon📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress
ConfigurationRelease-Alpha
Build Number31136
VersionPR #25302
Bundle IDorg.wordpress.alpha
Commit7c8b697
Installation URL359vubhl7ub10
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot
Copy link
Contributor

🤖 Build Failure Analysis

This build has failures. Claude has analyzed them - check the build annotations for details.

@sonarqubecloud
Copy link

@wpmobilebot
Copy link
Contributor

App Icon📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack
ConfigurationRelease-Alpha
Build Number31136
VersionPR #25302
Bundle IDcom.jetpack.alpha
Commit7c8b697
Installation URL1pe0dnvp8isog
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants