Skip to content

Break out member access denied email to separate mailer

Doug Stull requested to merge 466562-move-access-denied-email into master

What does this MR do and why?

Break out member access denied email to separate mailer

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

no change

source visibility level/format before after
public html Screenshot_2024-09-16_at_4.39.01_PM no change
public text Screenshot_2024-09-16_at_4.39.11_PM no change
private html Screenshot_2024-09-25_at_5.38.01_PM no change
private text Screenshot_2024-09-26_at_8.59.54_AM Screenshot_2024-09-25_at_5.38.09_PM

How to set up and validate locally

  1. restart gdk after switching to this branch(mailer preview won't refresh w/o that).
  2. Create a public group.
  3. In rails console, create an access request member
group = Group.last
require 'factory_bot'
FactorBot.create(:group_member, :access_request, source: group)
  1. Create a private group.
  2. In rails console, create an access request member
group = Group.last
require 'factory_bot'
FactorBot.create(:group_member, :access_request, source: group)
  1. use rails preview at this route: http://gdk.test:3000/rails/mailers/members/access_denied_mailer/

Related to #466562 (closed)

Edited by Doug Stull

Merge request reports

Loading