Skip to content

fix: convert http:// links to https:// in docs#1449

Open
memosr wants to merge 1 commit into
base:masterfrom
memosr:fix/http-to-https-docs-links
Open

fix: convert http:// links to https:// in docs#1449
memosr wants to merge 1 commit into
base:masterfrom
memosr:fix/http-to-https-docs-links

Conversation

@memosr
Copy link
Copy Markdown

@memosr memosr commented May 14, 2026

Summary

Converts 8 instances of http:// URLs to https:// across 5 documentation files to prevent mixed-content warnings and ensure secure connections to external and internal Base resources.

Changes

File Link
docs/terms-of-service.mdx Privacy Policy link (line 114)
docs/base-account/basenames/basenames-faq.mdx 3 links (lines 28, 33, 85)
docs/apps/builder-codes/app-developers.mdx 2 base.dev links (line 9)
docs/base-account/more/troubleshooting/usage-details/wallet-library-support.mdx ThirdWeb portal link
docs/get-started/base-mentorship-program.mdx X/Twitter profile link

Why

  • Mixed-content warnings in browsers can erode trust, especially in legal documents like the Terms of Service
  • All target domains (coinbase.com, base.org, base.dev, basename.app, thirdweb.com, x.com) force HTTPS via redirect anyway — using https:// directly avoids unnecessary 301 redirects
  • Consistency: most other links in the same files already use https://

Testing

No content changes — only protocol prefix. All target URLs verified accessible via HTTPS.

Updates 8 instances of http:// URLs to https:// across 5 documentation files
to prevent mixed-content warnings and ensure secure connections to external
and internal Base resources.

Files updated:
- docs/terms-of-service.mdx (Privacy Policy link)
- docs/base-account/basenames/basenames-faq.mdx (3 links)
- docs/apps/builder-codes/app-developers.mdx (2 base.dev links)
- docs/base-account/more/troubleshooting/usage-details/wallet-library-support.mdx (ThirdWeb)
- docs/get-started/base-mentorship-program.mdx (X/Twitter link)
@cb-heimdall
Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/2
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 1
Sum 2

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.

2 participants