Skip to content

Add a new OpenGraph tag fediverse:creator - #253

Merged
shugo merged 1 commit into
tdiary:masterfrom
shugo:feature/add-fediverse-creator
Nov 20, 2025
Merged

shugo merged 1 commit into
tdiary:masterfrom
shugo:feature/add-fediverse-creator

Conversation

@shugo

@shugo shugo commented Nov 20, 2025

Copy link
Copy Markdown
Member

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for the fediverse:creator OpenGraph tag, allowing blog authors to specify their Fediverse account ID. This follows Mastodon's standard for highlighting journalism content.

Key changes:

  • Added configuration option for ogp.fediverse.creator
  • Refactored configuration saving logic to use iteration instead of duplication
  • Added UI field for Fediverse User ID in configuration panel

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread plugin/ogp.rb Outdated
<h3>Facebook User ID</h3>
<p><input name="ogp.facebook.admins" value="#{h(@conf['ogp.facebook.admins'])}"></p>

<h3>Fediverse User ID</h3>

Copilot AI Nov 20, 2025

Copy link

Choose a reason for hiding this comment

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

[nitpick] The label 'Fediverse User ID' is inconsistent with the documented configuration key comment. The comment at line 9 refers to it as 'your fediverse ID', while similar labels for Facebook use 'Application ID' and 'User ID'. Consider changing this to 'Fediverse ID' for consistency with the comment, or update the comment to match this label.

Suggested change
<h3>Fediverse User ID</h3>
<h3>Fediverse ID</h3>

Copilot uses AI. Check for mistakes.
@shugo
shugo force-pushed the feature/add-fediverse-creator branch from b08ea4a to 16e58e1 Compare November 20, 2025 04:46
@shugo
shugo merged commit 6976afc into tdiary:master Nov 20, 2025
4 checks passed
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.

2 participants