Skip to content

fix(docs): add noopener to verify redirect example#1679

Open
maho0638 wants to merge 1 commit into
base:masterfrom
maho0638:fix-window-open-noopener
Open

fix(docs): add noopener to verify redirect example#1679
maho0638 wants to merge 1 commit into
base:masterfrom
maho0638:fix-window-open-noopener

Conversation

@maho0638

@maho0638 maho0638 commented Jul 1, 2026

Copy link
Copy Markdown

Summary

Adds noopener,noreferrer to the window.open(..., '_blank') example in the Base Verify redirect flow.

Why

The current example opens a new browsing context without explicitly preventing opener access. Adding noopener,noreferrer makes the copied example safer for production use.

Fixes #1470.

Test plan

  • Verified this PR only updates docs/base-account/guides/verify-social-accounts.mdx
  • Confirmed the code example still opens the Base Verify deep link in a new tab/window

@cb-heimdall

Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[docs] Add noopener/noreferrer to window.open example

2 participants