Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
marvinl803 wants to merge1 commit intopython:main
base:main
Choose a base branch
Loading
frommarvinl803:main

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 atboxes/box/40/change/ though depending on what@ewdurbin thinks.

@hugovk
Copy link
Member

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 usinghttps://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

We could make the relevant changes atboxes/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 freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@ewdurbinewdurbinAwaiting requested review from ewdurbinewdurbin is a code owner

@JacobCoffeeJacobCoffeeAwaiting requested review from JacobCoffeeJacobCoffee is a code owner

At least 0 approving reviews are required to merge this pull request.

Assignees
No one assigned
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4 participants
@marvinl803@JacobCoffee@hugovk@ewdurbin

[8]ページ先頭

©2009-2025 Movatter.jp