Skip to content

Resolve content link/profile presentation from the local window#25707

Merged
crazytonyli merged 8 commits into
trunkfrom
task/activity-content-local-window
Jun 26, 2026
Merged

Resolve content link/profile presentation from the local window#25707
crazytonyli merged 8 commits into
trunkfrom
task/activity-content-local-window

Conversation

@crazytonyli

@crazytonyli crazytonyli commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Note

I recommend reviewing commit by commit.

Instead of using UIViewController.topViewController which tries the global window instance, we should access the window based on the local context.

@dangermattic

dangermattic commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator
1 Warning
⚠️ View files have been modified, but no screenshot or video is included in the pull request. Consider adding some for clarity.

Generated by 🚫 Danger

@crazytonyli crazytonyli force-pushed the task/activity-content-local-window branch from b4e1878 to 1be09c4 Compare June 26, 2026 08:56
@crazytonyli crazytonyli requested a review from jkmassel June 26, 2026 08:59
@crazytonyli crazytonyli added this to the 27.1 milestone Jun 26, 2026
@crazytonyli crazytonyli marked this pull request as ready for review June 26, 2026 09:00
@crazytonyli crazytonyli enabled auto-merge June 26, 2026 09:00
@wpmobilebot

wpmobilebot commented Jun 26, 2026

Copy link
Copy Markdown
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 Number32856
VersionPR #25707
Bundle IDorg.wordpress.alpha
Commitefb3015
Installation URL5kjhd7b8v3ejo
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot

wpmobilebot commented Jun 26, 2026

Copy link
Copy Markdown
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 Number32856
VersionPR #25707
Bundle IDcom.jetpack.alpha
Commitefb3015
Installation URL3gqrau23akg7g
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@crazytonyli crazytonyli added this pull request to the merge queue Jun 26, 2026
Merged via the queue into trunk with commit ac007fa Jun 26, 2026
22 checks passed
@crazytonyli crazytonyli deleted the task/activity-content-local-window branch June 26, 2026 18:10
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