Skip to content

Update X social links from supermemoryai to supermemory#607

Merged
graphite-app[bot] merged 1 commit into
mainfrom
12-05-fix_x_handle_links
Dec 7, 2025
Merged

Update X social links from supermemoryai to supermemory#607
graphite-app[bot] merged 1 commit into
mainfrom
12-05-fix_x_handle_links

Conversation

@MaheshtheDev
Copy link
Copy Markdown
Member

@MaheshtheDev MaheshtheDev commented Dec 6, 2025

we recently got new x/twitter handle and update the references for the links

@MaheshtheDev MaheshtheDev marked this pull request as ready for review December 6, 2025 07:27
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Dec 6, 2025

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
supermemory-app 89d056e Dec 07 2025, 03:50 AM

Copy link
Copy Markdown
Member Author


How to use the Graphite Merge Queue

Add the label Main to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@claude
Copy link
Copy Markdown

claude Bot commented Dec 6, 2025

PR Review: Fix X Handle Links

Overview

This PR updates the X/Twitter handle from @supermemoryai to @supermemory across 4 files. The changes are straightforward and accomplish the stated goal.


✅ Code Quality & Best Practices

Positive aspects:

  • Consistent updates: All references have been updated systematically across documentation and code
  • Simple, focused change: The PR does exactly what it says with no scope creep
  • Proper external link handling: Uses target="_blank" and rel="noopener noreferrer" where appropriate for security

Minor observation:

  • The formatting change in apps/web/app/(navigation)/settings/support/page.tsx (lines 70-72) is a nice cleanup that improves readability

🔍 Potential Issues

No significant issues found, but here are some minor considerations:

  1. Missing newline at EOF: The file apps/web/app/(navigation)/settings/support/page.tsx previously was missing a newline at EOF, which has been fixed. Good catch!

  2. Consider a global constant: While not necessary for this change, if X/Twitter links appear in many places, consider defining the handle as a constant in a shared config file to make future updates easier:

    // e.g., in a config file
    export const SOCIAL_LINKS = {
      twitter: 'https://x.com/supermemory',
      github: 'https://github.com/supermemoryai',
      // ...
    }

    This is NOT a blocker for this PR, just a potential future improvement.


⚡ Performance Considerations

No performance impact: This is purely a string replacement with zero performance implications.


🔒 Security Concerns

Security is properly handled:

  • External links in apps/web/components/views/integrations.tsx correctly use target="_blank" with rel="noopener noreferrer" to prevent tab-nabbing attacks
  • window.open() calls with "_blank" are safe for the use case

🧪 Test Coverage

Testing considerations:

  • This change is low-risk and involves static links
  • Manual testing recommended: Click each updated link to verify the new handle is correct
  • No automated tests needed for this type of change

Verification checklist:

  • Click X/Twitter link in README
  • Click X/Twitter link in docs
  • Click both X/Twitter links in support page
  • Click X/Twitter link in integrations view
  • Verify all links open the correct profile (@SuperMemory)

📝 Summary

Recommendation: ✅ APPROVE

This is a clean, well-executed PR that does exactly what it needs to do. The changes are:

  • Consistent across all files
  • Free of bugs or security issues
  • Properly formatted
  • Low risk

No blocking issues identified. Great job on the straightforward update! 🎉


Adherence to Repository Standards

Per the CLAUDE.md guidelines:

  • ✅ Changes follow TypeScript standards
  • ✅ Code is properly formatted (Biome would approve)
  • ✅ No over-engineering or unnecessary abstractions
  • ✅ Focused change without scope creep

@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Dec 7, 2025

Merge activity

@Dhravya Dhravya changed the title fix: x handle links Update X social links from supermemoryai to supermemory Dec 7, 2025
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Dec 7, 2025

Movie gif. Sacha Baron Cohen as Borat sits in front of a map and raises his two thumbs, waggling his eyebrows and giving an approving nod and toothy grin. (Added via Giphy)

we recently got new x/twitter handle and update the references for the links
@graphite-app graphite-app Bot force-pushed the 12-05-fix_x_handle_links branch from 0477832 to 89d056e Compare December 7, 2025 03:44
@graphite-app graphite-app Bot merged commit 89d056e into main Dec 7, 2025
4 of 6 checks passed
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