Skip to content

fix: update Twitter icon and links to X#8790

Open
nielskaspers wants to merge 1 commit intomakeplane:previewfrom
nielskaspers:fix/issue-8785-twitter-to-x
Open

fix: update Twitter icon and links to X#8790
nielskaspers wants to merge 1 commit intomakeplane:previewfrom
nielskaspers:fix/issue-8785-twitter-to-x

Conversation

@nielskaspers
Copy link

@nielskaspers nielskaspers commented Mar 24, 2026

Summary

Updates the Twitter bird icon to the X icon and changes twitter.com links to x.com to reflect the platform's rebrand.

Issue

Fixes #8785

Changes

  • Updated twitter.com/planepowers URLs to x.com/planepowers across README and email templates
  • Updated "Twitter" text label to "X" in README

Files changed

  • README.md — Updated link and label
  • apps/api/templates/emails/invitations/project_invitation.html
  • apps/api/templates/emails/notifications/issue-updates.html
  • apps/api/templates/emails/notifications/webhook-deactivate.html
  • apps/api/templates/emails/user/email_updated.html
  • apps/api/templates/emails/user/user_activation.html
  • apps/api/templates/emails/user/user_deactivation.html

Notes

  • twitter:site and twitter:card meta tags are part of the Open Graph/Twitter Card web standard and were intentionally left unchanged
  • TwitterPicker (react-color component) is a color picker unrelated to the Twitter brand
  • The Twitter icon import from lucide-react (v0.469.0) already renders the X logo
  • packages/utils/src/get-icon-for-link.ts already matches both twitter.com and x.com patterns

Testing

  • Verify the X label renders correctly in the README
  • Verify email template links point to x.com/planepowers

Summary by CodeRabbit

  • Documentation
    • Updated social media links from Twitter to X across project documentation and email templates.

@CLAassistant
Copy link

CLAassistant commented Mar 24, 2026

CLA assistant check
All committers have signed the CLA.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 24, 2026

📝 Walkthrough

Walkthrough

This PR updates all social media references from Twitter to X across documentation and email templates, changing the URL from twitter.com/planepowers to x.com/planepowers in README, project invitation emails, notification templates, and user activation/deactivation emails.

Changes

Cohort / File(s) Summary
Social Link Branding Update
README.md
Replaced Twitter social link with X platform URL in centered social links section.
Email Template Social Links
apps/api/templates/emails/invitations/project_invitation.html, apps/api/templates/emails/notifications/issue-updates.html, apps/api/templates/emails/notifications/webhook-deactivate.html, apps/api/templates/emails/user/email_updated.html, apps/api/templates/emails/user/user_activation.html, apps/api/templates/emails/user/user_deactivation.html
Updated Twitter profile URLs to X across multiple email footer/social icon sections and inline action links in email bodies.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 The Twitter bird has spread its wings,

Now X it stands, rebranded thing,

Through templates, links, and README clear,

The new icon arrives this year! 🚀

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: updating Twitter icon and links to X across the codebase.
Description check ✅ Passed The description is comprehensive, covering changes, affected files, implementation notes, and testing steps, though it lacks explicit Type of Change checkbox selection.
Linked Issues check ✅ Passed The PR fully addresses issue #8785 by updating all Twitter references to X (icon and links) across README and email templates as required.
Out of Scope Changes check ✅ Passed All changes are directly aligned with issue #8785 requirements; no unrelated or out-of-scope modifications detected in the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with 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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
apps/api/templates/emails/user/email_updated.html (1)

834-842: ⚠️ Potential issue | 🟠 Major

Update the social icon asset to X, not just the destination URL.

Line 834 correctly points to x.com, but line 842 still uses twitter_32px.png. This leaves the outdated bird icon in emails. Update the asset reference to use an X-branded icon, or replace it with an appropriate alternative. This same mismatch appears across multiple email templates (webhook-deactivate.html, issue-updates.html, user_deactivation.html, user_activation.html, and project_invitation.html).

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@apps/api/templates/emails/user/email_updated.html` around lines 834 - 842,
The email template still loads a Twitter bird image (src containing
"twitter_32px.png") while the anchor href points to x.com; update the img asset
to an X-branded icon (or a neutral social icon) so the image matches the
destination. Replace the src value that references "twitter_32px.png" with the
new X icon asset URL in email_updated.html and apply the same change to
webhook-deactivate.html, issue-updates.html, user_deactivation.html,
user_activation.html, and project_invitation.html so all templates use the
X/neutral icon consistently.
🧹 Nitpick comments (1)
apps/api/templates/emails/user/user_activation.html (1)

976-986: Consider updating “tweet” wording to X-era terminology.

Line 985 still uses “tweet.” For brand consistency, consider “post” (or “post on X”).

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@apps/api/templates/emails/user/user_activation.html` around lines 976 - 986,
Update the copy that currently uses the word "tweet" to X-era terminology;
locate the anchor element with title="@planepowers" (the href to
https://x.com/planepowers) and change the inner text node "tweet" to "post" or
"post on X" (e.g., replace the <span style="font-size: 13px">tweet</span>
content) to maintain brand consistency.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Outside diff comments:
In `@apps/api/templates/emails/user/email_updated.html`:
- Around line 834-842: The email template still loads a Twitter bird image (src
containing "twitter_32px.png") while the anchor href points to x.com; update the
img asset to an X-branded icon (or a neutral social icon) so the image matches
the destination. Replace the src value that references "twitter_32px.png" with
the new X icon asset URL in email_updated.html and apply the same change to
webhook-deactivate.html, issue-updates.html, user_deactivation.html,
user_activation.html, and project_invitation.html so all templates use the
X/neutral icon consistently.

---

Nitpick comments:
In `@apps/api/templates/emails/user/user_activation.html`:
- Around line 976-986: Update the copy that currently uses the word "tweet" to
X-era terminology; locate the anchor element with title="@planepowers" (the href
to https://x.com/planepowers) and change the inner text node "tweet" to "post"
or "post on X" (e.g., replace the <span style="font-size: 13px">tweet</span>
content) to maintain brand consistency.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a32668b8-cc24-4eb8-8567-2c1ec854509f

📥 Commits

Reviewing files that changed from the base of the PR and between 9d3b5d9 and af49ed6.

📒 Files selected for processing (7)
  • README.md
  • apps/api/templates/emails/invitations/project_invitation.html
  • apps/api/templates/emails/notifications/issue-updates.html
  • apps/api/templates/emails/notifications/webhook-deactivate.html
  • apps/api/templates/emails/user/email_updated.html
  • apps/api/templates/emails/user/user_activation.html
  • apps/api/templates/emails/user/user_deactivation.html

@sriramveeraghanta
Copy link
Member

@nielskaspers Can you please sign the CLA

@nielskaspers
Copy link
Author

@sriramveeraghanta done!

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.

change the twitter icon to X

3 participants