Skip to content

Email rework, cta_link, and new design for confirmation email

Anthony Shull requested to merge as/feat/935/confirmation-email into develop

I rewrote a lot of notifications and every mailer execept recsys. Cleaned up a ton of redundant code as well as confusing uses of variables. I also added subject and cta_link as fields on the notification. They get created before save so that they can just be read out when the notification is actually sent.

There are too many emails to test each manually here. But, we'll highlight some.

https://www.loom.com/share/74539a464a2f448592190d3f45cf854f

%> bundle exec rails db:reset

Closes https://gitlab.com/JOGL/JOGL/-/issues/935 Closes https://gitlab.com/JOGL/JOGL/-/issues/828 Closes https://gitlab.com/JOGL/JOGL/-/issues/182

Edited by Anthony Shull

Merge request reports