Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Twitter Icon to New X Icon #19

Closed
wants to merge 1 commit into from
Closed

Update Twitter Icon to New X Icon #19

wants to merge 1 commit into from

Conversation

BishalJena
Copy link

@BishalJena BishalJena commented Aug 1, 2024

This pull request addresses issue #18, which involves updating the old Twitter bird icon to the new “X” icon. The following changes have been made:

•	Replaced the SVG path data in the TwitterIcon component with the new “X” logo.
•	Updated relevant component and file names to reflect the change from “Twitter” to “X,” where applicable.

This change is part of the rebranding from Twitter to X and ensures that all visual representations align with the new branding guidelines.
Fixes: #18

Copy link

vercel bot commented Aug 1, 2024

@BishalJena is attempting to deploy a commit to the OpenJS Foundation Team on Vercel.

A member of the Team first needs to authorize it.

@BishalJena
Copy link
Author

@ibgreen @chrisgervang kindly check this PR and merge if passes the checks.

@ibgreen
Copy link
Collaborator

ibgreen commented Aug 1, 2024

@BishalJena Thanks for the PR. Not sure I have the authority to authorize the Vercel deployment though.

@BishalJena
Copy link
Author

BishalJena commented Aug 1, 2024

@ibgreen can you ping other maintainers who'd have the vercel authorization? i couldn't find anyone else active except you.

@BishalJena
Copy link
Author

@chrisgervang as you reviewed the changes, can you authorize vercel and merge this PR?

@BishalJena BishalJena closed this by deleting the head repository Aug 7, 2024
@chrisgervang
Copy link
Collaborator

Hey @BishalJena, I have permission to merge this but I'm not familiar with the deployment system and ran into an issue.

Any particular reason to close this?

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.

Update Twitter Links to X
3 participants