Skip to content

Refactor email sending flow to use Phoenix.Components instead of Phoenix.Swoosh#220

Open
DimitrijeDimitrijevic wants to merge 1 commit into
ClaperCo:devfrom
DimitrijeDimitrijevic:phoenix_swoosh_replacement
Open

Refactor email sending flow to use Phoenix.Components instead of Phoenix.Swoosh#220
DimitrijeDimitrijevic wants to merge 1 commit into
ClaperCo:devfrom
DimitrijeDimitrijevic:phoenix_swoosh_replacement

Conversation

@DimitrijeDimitrijevic

Copy link
Copy Markdown
Contributor

Replaces the separate HEEx template files with Phoenix function components for email rendering.

Changes

  • Consolidate email templates into NotifierComponents module with inline HEEx
  • Extract shared layout to EmailLayout module
  • Reorganize user notifier: user_notifier.ex → user/notifier.ex + user/notifier_components.ex
  • Remove lib/claper_web/templates/user_notifier/ and lib/claper_web/templates/leader_notifier/

Benefits

  • Easier to maintain - templates live alongside their rendering logic
  • Better code organization with shared layout component
  • Follows modern Phoenix/Swoosh patterns

If you have questions, please let me know.

@stale

stale Bot commented May 25, 2026

Copy link
Copy Markdown

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale Bot added the wontfix This will not be worked on label May 25, 2026
@stale stale Bot closed this Jun 1, 2026
@DimitrijeDimitrijevic

Copy link
Copy Markdown
Contributor Author

Hi @alxlion, why this got closed ?
Did you have a chance to take a look at this ?

Thanks!

@alxlion alxlion reopened this Jun 28, 2026
@stale stale Bot removed the wontfix This will not be worked on label Jun 28, 2026
@alxlion

alxlion commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

Hi @alxlion, why this got closed ?

Did you have a chance to take a look at this ?

Thanks!

Sorry the bot closed this, still in review

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