-
-
Notifications
You must be signed in to change notification settings - Fork 120
Remove Twitter links in English version #808
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
Conversation
✅ Deploy Preview for numpy-org ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Thanks for looking into this. xref numpy/numpy#28036 (twitter is in the NumPy readme). We removed the main X/twitter link in #767 (we never used it that much), so we may even just want to remove it. @InessaPawson what do you prefer? |
@seberg +1 on removing the mention and the link to the NumPy X account. |
@star1327p Thank you for working on this PR! Please note that the translations on numpy.org are done by our translations team via Crowdin. Please remove all pending changes for any language versions other than English. |
@seberg @InessaPawson Thank you both for reviewing! I have removed the Twitter link in the English version, and reverted my changes in the other languages. |
Thank you, @star1327p! |
Brief description of what is fixed or changed
Replaced Twitter with X, including the links https://x.com/numpy_team .
Updated the Youtube channel to a shorter URL https://www.youtube.com/@NumPy_team .