Skip to content

feat: email signature#128

Open
thaninbew wants to merge 1 commit into
mainfrom
114-email-signature
Open

feat: email signature#128
thaninbew wants to merge 1 commit into
mainfrom
114-email-signature

Conversation

@thaninbew
Copy link
Copy Markdown
Collaborator

Description

Wired up the frontend Email Editor to the backend template endpoints and fixed the signature/CTA state loading.

Changes Made

  • Frontend changes
  • Added template fetching on mount for all 3 email types.
  • Fixed the duplicating signature bug on reload by wrapping metadata in hidden HTML comments to safely parse out the body text.
  • Added all missing signature inputs (Name, Email, Pronouns, Socials, etc.) and updated defaults to the official FCC links.

Testing & Verification

  • Manual testing completed
  • Lint passes

Future Improvements/Notes

  • Left a placeholder text input for the signature image URL for now. I'm waiting on the S3/CDN feature from Hannah's ticket before swapping this to a real input.

Related Issues

Closes #114

@thaninbew thaninbew requested review from aaronashby and piersolh May 30, 2026 08:38
@thaninbew thaninbew linked an issue May 30, 2026 that may be closed by this pull request
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.

Email Signature

1 participant