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

Fedi credit #730

Merged
merged 4 commits into from
Oct 30, 2024
Merged

Fedi credit #730

merged 4 commits into from
Oct 30, 2024

Conversation

mirisuzanne
Copy link
Member

@mirisuzanne mirisuzanne commented Oct 30, 2024

Description

Two parts:

  • Adds bluesky social links
  • Adds fediverse:creator metadata to our open-graph tags

The latter only has any impact for accounts that set up oddsite as an approved source in their mastodon profiles. I also don't want to post social links if people don't want them - so assigning all of you where I added data. Feel free to remove accounts you'd prefer not to list here.

Steps to test/reproduce

  • Social media links for the company (site footer and about page)
  • For individuals who are on masto/bluesky, on the about page or author pages
  • The document head open graph data, especially on posts by those people

Copy link

View diff of compiled files (may take a few minutes): https://github.com/oddbird/oddleventy-built/compare/main..fedi-credit

Copy link
Contributor

@jamesnw jamesnw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

Copy link
Member

@stacyk stacyk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just created a profile. Not sure if I will use it but I would like to get off of twitter and I don't love the UX of mastodon.

Co-authored-by: Stacy Kvernmo <[email protected]>
@mirisuzanne mirisuzanne merged commit 3358b7a into main Oct 30, 2024
10 checks passed
@mirisuzanne mirisuzanne deleted the fedi-credit branch October 30, 2024 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants