fixed twitter username and some other fixes#1203
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
WalkthroughProject-wide URL updates migrate references from peanut.to to peanut.me across README, metadata, redirects, config defaults, footer/socials, profile/setup links, and tooltips. One Twitter handle changes to @joinpeanut. No logic or exported API changes. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Pre-merge checks (2 passed, 1 inconclusive)❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
Tip 👮 Agentic pre-merge checks are now available in preview!Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.
Please see the documentation for more information. Example: reviews:
pre_merge_checks:
custom_checks:
- name: "Undocumented Breaking Changes"
mode: "warning"
instructions: |
Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).Please share your feedback with us on this Discord post. ✨ Finishing touches
🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (3)
src/config/wagmi.config.tsx (1)
34-36: Avoid undefined favicon when NEXT_PUBLIC_BASE_URL is unsetDerive icons from the same base URL fallback to prevent
undefined/favicon.icolocally/preview.-const metadata = { +const baseUrl = process.env.NEXT_PUBLIC_BASE_URL || 'https://peanut.me' +const metadata = { name: 'Peanut Protocol', description: 'Peanut protocol - send crypto with links', - url: process.env.NEXT_PUBLIC_BASE_URL || 'https://peanut.me', // origin must match your domain & subdomain - icons: [`${process.env.NEXT_PUBLIC_BASE_URL}/favicon.ico`], + url: baseUrl, // origin must match your domain & subdomain + icons: [`${baseUrl}/favicon.ico`], }public/game/peanut-game.html (1)
7-7: Use canonical HTTPS URL (and correct path) for OGPrefer
httpsand the exact page URL to avoid 301 chains and improve sharing.-<meta property="og:url" content="http://www.peanut.me" /> +<meta property="og:url" content="https://peanut.me/game/peanut-game.html" />README.md (1)
22-22: Typo: duplicated wordChange “secure secure context” → “secure context”.
-# Note: run pnpm run dev:https if you need to work in a secure secure context +# Note: run pnpm run dev:https if you need to work in a secure context
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (8)
README.md(1 hunks)public/game/peanut-game.html(1 hunks)redirects.json(1 hunks)src/components/Global/Footer/consts.ts(3 hunks)src/components/Profile/index.tsx(2 hunks)src/components/Setup/Views/SetupPasskey.tsx(1 hunks)src/config/wagmi.config.tsx(1 hunks)src/constants/tooltips.ts(1 hunks)
🧰 Additional context used
🧠 Learnings (3)
📓 Common learnings
Learnt from: Hugo0
PR: peanutprotocol/peanut-ui#1014
File: src/components/Claim/Link/Initial.view.tsx:413-413
Timestamp: 2025-07-24T13:26:10.290Z
Learning: In the peanut-ui repository, the change from `${SQUID_API_URL}/route` to `${SQUID_API_URL}/v2/route` in src/components/Claim/Link/Initial.view.tsx was a typo fix, not an API migration, as the codebase was already using Squid API v2.
📚 Learning: 2025-07-24T13:26:10.290Z
Learnt from: Hugo0
PR: peanutprotocol/peanut-ui#1014
File: src/components/Claim/Link/Initial.view.tsx:413-413
Timestamp: 2025-07-24T13:26:10.290Z
Learning: In the peanut-ui repository, the change from `${SQUID_API_URL}/route` to `${SQUID_API_URL}/v2/route` in src/components/Claim/Link/Initial.view.tsx was a typo fix, not an API migration, as the codebase was already using Squid API v2.
Applied to files:
src/components/Profile/index.tsxsrc/components/Setup/Views/SetupPasskey.tsxsrc/components/Global/Footer/consts.tssrc/config/wagmi.config.tsxsrc/constants/tooltips.ts
📚 Learning: 2025-06-18T19:56:55.443Z
Learnt from: jjramirezn
PR: peanutprotocol/peanut-ui#919
File: src/components/Withdraw/views/Initial.withdraw.view.tsx:87-87
Timestamp: 2025-06-18T19:56:55.443Z
Learning: In withdraw flows for Peanut Wallet, the PeanutActionDetailsCard should always display "USDC" as the token symbol because it shows the amount being withdrawn from the Peanut Wallet (which holds USDC), regardless of the destination token/chain selected by the user. The TokenSelector is used for choosing the withdrawal destination, not the source display.
Applied to files:
src/constants/tooltips.ts
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Deploy-Preview
🔇 Additional comments (10)
redirects.json (2)
4-4: Confirm redirect permanence for /docsIf the domain migration is final, consider
"permanent": truefor SEO/cache benefits. Otherwise, keeping it non-permanent is fine.
19-31: Validate legacy.peanut.to redirectsredirects.json still contains entries pointing to https://legacy.peanut.to (lines 19–31). Confirm this is intentional and whether those legacy hosts will migrate to .me; if not, update the redirects.
Also found other peanut.to references — review for consistency: src/middleware.ts, src/app/api/health/route.ts, src/components/Payment/Views/Error.validation.view.tsx, src/components/Global/WalletNavigation/index.tsx, scripts/README-qr-generator.md, src/assets/chains/arbitrum.svg.
src/components/Setup/Views/SetupPasskey.tsx (1)
107-107: LGTM: docs link updated to .meMatches the project-wide domain change.
public/game/peanut-game.html (1)
9-10: LGTM: Twitter handle updatedBoth twitter:site and twitter:creator now point to @joinpeanut.
src/components/Global/Footer/consts.ts (3)
6-6: LGTM: Twitter URL updated to @joinpeanut
21-21: LGTM: Docs (gitbook) moved to docs.peanut.me
34-34: LGTM: Docs link updated in LINKSREADME.md (1)
1-1: LGTM: “Live at” display moved to .mesrc/components/Profile/index.tsx (2)
49-49: LGTM: Referrals docs link updated
71-71: LGTM: Fees docs link updated
| For US bank accounts, enter just your bank account number, no routing number. | ||
|
|
||
| <a href="https://docs.peanut.to/cashout/supported-geographies" target="_blank" class="underline text-blue-600">Supported regions</a>`, | ||
| <a href="https://docs.peanut.me/cashout/supported-geographies" target="_blank" class="underline text-blue-600">Supported regions</a>`, |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Add rel="noopener noreferrer" on target="_blank" links (prevent tabnabbing)
These tooltip anchors open new tabs; add rel attribute for security/perf parity with other components (e.g., SetupPasskey).
-<a href="https://docs.peanut.me/cashout/supported-geographies" target="_blank" class="underline text-blue-600">Supported regions</a>
+<a href="https://docs.peanut.me/cashout/supported-geographies" target="_blank" rel="noopener noreferrer" class="underline text-blue-600">Supported regions</a>-<a href="https://docs.peanut.me/cashout/supported-geographies" target="_blank" class="underline text-blue-600">Learn more about supported regions</a>
+<a href="https://docs.peanut.me/cashout/supported-geographies" target="_blank" rel="noopener noreferrer" class="underline text-blue-600">Learn more about supported regions</a>Also applies to: 14-14
🤖 Prompt for AI Agents
In src/constants/tooltips.ts around lines 6 and 14, the anchor tags that use
target="_blank" lack rel="noopener noreferrer"; update both anchors to include
rel="noopener noreferrer" (matching other components like SetupPasskey) to
prevent tabnabbing and ensure consistent security/performance behavior when
opening links in a new tab.
2 participants
We changed to @joinpeanut
Fixed the LP links