Skip to content

Comments

Remove Mastodon link from navigation.json closes #7873#8645

Closed
skyeyeye wants to merge 1 commit intonodejs:mainfrom
skyeyeye:patch-2
Closed

Remove Mastodon link from navigation.json closes #7873#8645
skyeyeye wants to merge 1 commit intonodejs:mainfrom
skyeyeye:patch-2

Conversation

@skyeyeye
Copy link

@skyeyeye skyeyeye commented Feb 20, 2026

Removed Mastodon link from navigation.

Description

Validation

Related Issues

#7873

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run pnpm format to ensure the code follows the style guide.
  • I have run pnpm test to check if all tests are passing.
  • I have run pnpm build to check if the website builds without errors.
  • I've covered new added functionality with unit tests if necessary.

Removed Mastodon link from navigation.

Signed-off-by: Ayush Saha <ayushsaha.work@gmail.com>
@skyeyeye skyeyeye requested a review from a team as a code owner February 20, 2026 07:51
@vercel
Copy link

vercel bot commented Feb 20, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
nodejs-org Ready Ready Preview Feb 20, 2026 7:53am

Request Review

@github-actions
Copy link
Contributor

👋 Codeowner Review Request

The following codeowners have been identified for the changed files:

Team reviewers: @nodejs/nodejs-website

Please review the changes when you have a chance. Thank you! 🙏

@AugustinMauroy
Copy link
Member

You should also remove the icon components

@skyeyeye
Copy link
Author

skyeyeye commented Feb 20, 2026

You should also remove the icon components

I am not able to do that,when I try to delete its giving error Vercel
Vercel — Deployment has failed
image
And this needs to be deleted right https://github.com/nodejs/nodejs.org/blob/main/packages/ui-components/src/Icons/Social/Mastodon.tsx?

@avivkeller avivkeller closed this Feb 20, 2026
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.

3 participants