Changed old twitter icon to X

[krushnarout]

Fixes Issue

Closes #9971

Changes proposed

Changed old twitter icon to new X icon.

Check List (Check all the applicable boxes)

• My code follows the code style of this project.
• My change requires changes to the documentation.
• I have updated the documentation accordingly.
• All new and existing tests passed.
• This PR does not contain plagiarized content.
• The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

[Sindhuinti]

Looks good to me 🚢

[adityaraute]

For those wondering what's the difference between the two

[kumarsonsoff3]

Changes seems a bit straight to me, but it gives the following error when we click on the share icon on someone's profile.

[krushnarout]

@kumarsonsoff3 I think you forget to add BsTwitterX on import { BsTwitter, BsFacebook, BsLinkedin } from "react-icons/bs"; while checking. Can you double check it?

[ChinmayMhatre]

Changes seems a bit straight to me, but it gives the following error when we click on the share icon on someone's profile.

This might be a separate issue. Can you share which page and which icon. maybe steps to reproduce?

[kumarsonsoff3]

This might be a separate issue. Can you share which page and which icon. maybe steps to reproduce?

I thought the same, but either this icon BsTwitter is used in some other module that needs update or something else.

• I tried undoing these changes, and that works fine.
• Might want to look at this clip:
  https://www.loom.com/share/f97ea2b6aaac4ea98f67223196ca4f08?sid=bac5b920-f154-45ee-a243-abb9f87a1dc0

[adityaraute]

This clip confirms that the PR creates issues that needs to be addressed.
The modal is probably bugging because of the missing icon. thanks for sharing @kumarsonsoff3

[krushnarout]

@adityaraute Can anyone tell me, how can I fix this ?

[adityaraute]

You have updated your branch and so there is no difference between your fork and this repo at the moment.
You'll have to once again commit your changes to your branch and then the PR will be mergeable.

[krushnarout]

@kumarsonsoff3 I've fixed the error, Please check and let me know any changes needed.

Screenshot

[kumarsonsoff3]

Looks awesome. Great collaboration everyone. 👍

[krushnarout]

Thank you everyone