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

Changed links on navbar from an outdated twitter page to a more recen… #2652

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

marvinl803
Copy link

This solves issue #2636.

I have updated the links in the sidebars to the official Python Software Foundation (PSF) Twitter account (twitter.com/thespf), which is currently active and better represents the official social media presence for Python. This ensures that the sidebar links now lead to more relevant, up-to-date resources for users.

@JacobCoffee
Copy link
Member

Thanks for the PR! I know it's confusing but box data is actually controlled via Django Admin and the fixture data is just exports of that for easy bootstrapping of development environments; this means that your PR would have no effect :\

We could make the relevant changes at boxes/box/40/change/ though depending on what @ewdurbin thinks.

@hugovk
Copy link
Member

hugovk commented Nov 7, 2024

The box is specifically about Python Insider so I suggest removing the old Twitter link.

Another option is to replace it with something else. It's easy to hook up an RSS feed to Mastodon using https://feedsin.space

I used it for @peps and @python_releases and it would be good to do one for Insider too.

@ewdurbin
Copy link
Member

ewdurbin commented Nov 7, 2024

We could make the relevant changes at boxes/box/40/change/ though depending on what @ewdurbin thinks.

Updating the boxes fixture in addition to making the change in Django admin for prod is a good way of documenting what's changed. So if we settle on a correct solution here, update the box in admin then merge the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants