fix(ui): correct external link target and add rel attributes

[Mayankaggarwal8055]

#64541

Problem

Additionally, NothingFoundInfo.tsx was using target="blank" instead of the correct target="_blank".

Changes

• Updated external link attributes to use target="_blank" with rel="noopener noreferrer"
• Added tests to verify correct link behavior

Result

• External links now open securely in a new tab
• Link attributes follow best practices

[Srabasti]

Hey @Mayankaggarwal8055 -
Static checks are failing. Can you run prek locally and then commit your branch to repo?
Static checks fail ho rahe hain. Kya aap apane local par prek chalaakar, phir apne branch ko repo mein commit kar sakte hain?

[potiuk]

@Mayankaggarwal8055 This PR has been converted to draft because it does not yet meet our Pull Request quality criteria.

Issues found:

• ❌ Pre-commit / static checks: Failing: CI image checks / Static checks. Run prek run --from-ref main locally to find and fix issues. See Pre-commit / static checks docs.
• ❌ Other failing CI checks: Failing: Basic tests / React UI tests, Additional PROD image tests / Chromium UI e2e tests with PROD image / Chromium UI e2e tests. Run prek run --from-ref main locally to reproduce. See static checks docs.

What to do next:

• The comment informs you what you need to do.
• Fix each issue, then mark the PR as "Ready for review" in the GitHub UI - but only after making sure that all the issues are fixed.
• There is no rush — take your time and work at your own pace. We appreciate your contribution and are happy to wait for updates.
• Maintainers will then proceed with a normal review.

Converting a PR to draft is not a rejection — it is an invitation to bring the PR up to the project's standards so that maintainer review time is spent productively. There is no rush — take your time and work at your own pace. We appreciate your contribution and are happy to wait for updates. If you have questions, feel free to ask on the Airflow Slack.

[Copilot]

Pull request overview

This PR fixes Airflow UI documentation links that were 404’ing when the running Airflow version didn’t have published docs (e.g., dev/unreleased versions), and corrects external-link behavior.

Changes:

• Point docs links to the stable documentation instead of version-specific docs.
• Fix external link attributes (target="_blank" + rel="noopener noreferrer").
• Add tests covering the updated link targets.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File	Description
airflow-core/src/airflow/ui/src/pages/Connections/NothingFoundInfo.tsx	Switch connection docs link to stable and fix external link attributes.
airflow-core/src/airflow/ui/src/pages/Connections/NothingFoundInfo.test.tsx	Add test asserting stable docs URL and correct link attributes.
airflow-core/src/airflow/ui/src/layouts/Nav/DocsButton.tsx	Switch “version” docs menu item to use stable docs index URL.
airflow-core/src/airflow/ui/src/layouts/Nav/DocsButton.test.tsx	Add test ensuring the version menu item points to the stable docs index URL.

[pierrejeambrun]

It appears that his is always using the 'stable' link and not the 'version' one anymore.

I don't think that's the correct approach.

Can you spin up a different PR for fixing the blank and other attributes link, so we can get that merged easily without waiting for resolution of the stable/version doc.

[Mayankaggarwal8055]

Thanks for the feedback!

I’ll run the pre-commit checks locally and fix all the failing issues. I’ll also address the test failures and update the PR accordingly.

Will push the updates soon and mark it ready for review once everything passes.

[potiuk]

The PR has different language description

[potiuk]

Please fix it - your PRs keep on having strange issues showing that you have not looked at them this is a warning sign that will make your PRs go by default in draft/close queue if it does not improve. If you use agents you are still responsible for the code you submit so be careful

[Mayankaggarwal8055]

Sorry @potuik for the issues earlier — I realize I should have reviewed the PR more carefully before submitting.

I’ve now run prek run --all-files locally and applied all the changes. However, one of the checks is still failing on CI, and I’m not sure why. At the moment, another check is still in progress.

and also updated the pr description to official language used.

I’ll keep investigating, but if there’s anything specific I might be missing, I’d really appreciate your guidance.

[github-actions]

Backport successfully created: v3-2-test

Note: As of Merging PRs targeted for Airflow 3.X
the committer who merges the PR is responsible for backporting the PRs that are bug fixes (generally speaking) to the maintenance branches.

In matter of doubt please ask in #release-management Slack channel.

Status	Branch	Result
✅	v3-2-test