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

Add metadata to New Follower E-Mail #1172

Merged
merged 15 commits into from
Feb 18, 2025
Merged

Add metadata to New Follower E-Mail #1172

merged 15 commits into from
Feb 18, 2025

Conversation

akirk
Copy link
Member

@akirk akirk commented Jan 16, 2025

This adds some metadata like the user icon and the user summary to the new follower e-mail and adds an action for other plugins to include their data (for example, a Follow Back link). See akirk/friends#428

Screenshot 2025-01-16 at 14 46 08

@obenland obenland force-pushed the enrich-follower-email branch from b0e1bae to f931b16 Compare February 17, 2025 15:14
@obenland obenland force-pushed the enrich-follower-email branch from 3bd7728 to f69a2d2 Compare February 17, 2025 15:34
@obenland obenland self-requested a review February 17, 2025 15:36
obenland
obenland previously approved these changes Feb 17, 2025
Copy link
Member

@obenland obenland left a comment

Choose a reason for hiding this comment

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

I think this is good to go now. @pfefferle Do you want to take another look?

@pfefferle
Copy link
Member

Shouldn't we add a link to the followers list to the footer?

@obenland
Copy link
Member

@pfefferle I added the link and made it a bit clearer who the new follower is following:

image

Copy link
Member

@pfefferle pfefferle left a comment

Choose a reason for hiding this comment

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

Works like a charm!

thanks @akirk and @obenland

@pfefferle pfefferle merged commit 3a913e8 into trunk Feb 18, 2025
11 checks passed
@pfefferle pfefferle deleted the enrich-follower-email branch February 18, 2025 14:55
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.

3 participants