vatsrahul1001 opened a new pull request, #72776:
URL: https://github.com/apache/airflow/pull/72776

   Backport of #72196 to `v3-3-test` for the 3.3.2 release.
   
   The legacy plugin views and the auth-manager security views (Users, Roles, 
…) render in an iframe, but the browser address bar stayed on the outer route 
while users navigated inside the frame. That broke deep linking: an inner page 
such as editing a specific user could not be bookmarked, shared, or reloaded, 
and Back/Forward did nothing. This mirrors the framed same-origin location to 
the address bar and reconstructs it on load so those pages become linkable 
again.
   
   **Backport adaptation:** the original `Iframe.tsx` diff sits on top of 
asset-iframe support (`assetId` / `useAssetServiceGetAsset`) that does not 
exist on `v3-3-test`. The URL-sync change itself is independent of that, so it 
was applied onto `v3-3-test`'s existing Iframe (add the `page` route param, 
`iframeRef`, `isNavView`, and the `useIframeUrlSync` wiring) without pulling in 
the unrelated asset code. `Security.tsx` keeps `v3-3-test`'s 
`src/components/ui` import path (main's `src/system-components` rename is not 
backported). The hook, tests, e2e spec, `router.tsx`, and `ExternalView.tsx` 
changes applied cleanly.
   
   Verified: `Iframe.test.tsx` + `Security.test.tsx` pass (7/7); `tsc --noEmit` 
clean.
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Code (Opus 4.8)
   
   Generated-by: Claude Code (Opus 4.8) following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to