Skip to content

pull request for issue #1#32

Open
aboodJS wants to merge 6 commits intomighty840:mainfrom
aboodJS:email-header
Open

pull request for issue #1#32
aboodJS wants to merge 6 commits intomighty840:mainfrom
aboodJS:email-header

Conversation

@aboodJS
Copy link
Copy Markdown

@aboodJS aboodJS commented Feb 22, 2026

Summary

this pull request solves the issue of the hardcoded email header in email.ts as described in issue #1

Changes

this pull request adds a function called getSiteName in email.ts that fetches the site name from the database using an environment variable, defaulting to "kitchenasty" if this is not configured

  • added an async function called getSiteName
  • replaced the hardcoded strings in the "h1" tags of the email templates with the returned value of the function

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Refactoring (no functional changes)

How to Test

  1. configure the siteName attribute in the sitesettings table
  2. set the environment variable to the siteName
  3. check if the value appears in the email

Checklist

  • My code follows the project's coding standards
  • I have added tests for my changes
  • All new and existing tests pass (npm test)
  • I have updated documentation if needed
  • My changes don't introduce new TypeScript errors (npm run lint)

@aboodJS aboodJS closed this Feb 23, 2026
@aboodJS aboodJS reopened this Feb 23, 2026
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.

1 participant