fix: add rel protection to footer policy link - #855
Conversation
Signed-off-by: Mridankan Mandal <xerontitan90@gmail.com>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
There was a problem hiding this comment.
Code Review
This pull request updates docusaurus.config.js by fixing a minor formatting issue and adding rel="noopener noreferrer" to an external link in the footer for security. There are no review comments, and I have no additional feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
Which issue(s) this PR fixes:
Fixes #853.
Bug fix.
The footer policy link opens a new tab with
target="_blank"but does not includerel="noopener noreferrer".The footer policy link now keeps the same behavior and adds
rel="noopener noreferrer"for the external new-tab anchor.No.
npx -y yarn@1.22.22 install --frozen-lockfile.npx -y -p node@20 --call "node ./node_modules/@docusaurus/core/bin/docusaurus.mjs build".npx -y prettier@3.3.3 --check docusaurus.config.js.Before the fix:
After the fix: