-
Notifications
You must be signed in to change notification settings - Fork 39
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
Replace twitter with mastodon and bluesky in documentation #553
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one minor change request, approving pre-emptively. Thanks @Czaki!
This seems a bit excessive 😆 |
We should also remove the twitter logo from the footer - I will do that in the theme |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Requesting changes so we don't merge the giant icon.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great to me, thanks @Czaki !
References and relevant issues
Description
As twitter/X does not allow browsing post without logging it, this PR replaces all mentions with mastodon and bluesky.
This PR does not remove the link from the footer. It looks like we need to edit our template for this.