Skip to content

LG-10393: Remove unneeded IPP outage email template#10544

Merged
aduth merged 1 commit intomainfrom
aduth-rm-ipp-outage-email
May 2, 2024
Merged

LG-10393: Remove unneeded IPP outage email template#10544
aduth merged 1 commit intomainfrom
aduth-rm-ipp-outage-email

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented May 2, 2024

🎫 Ticket

LG-10393 (clean-up task after LG-9901)

🛠 Summary of changes

Removes the email template for in_person_outage_notification. This was never referenced in code, but was used as part of a one-time roll plan (ROLL-0011) that was completed in May 2023.

📜 Testing Plan

Build should pass.

changelog: Internal, Code Quality, Remove unused code
@aduth aduth requested review from a team and n1zyy May 2, 2024 14:47
@svalexander
Copy link
Contributor

Can we reference this ticket with this pr? we actually had one in the backlog for this work

Copy link
Contributor

@svalexander svalexander left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we update pr name to reference ticket mentioned in comment?

@aduth aduth changed the title Remove unneeded IPP outage email template LG-10393: Remove unneeded IPP outage email template May 2, 2024
@aduth
Copy link
Contributor Author

aduth commented May 2, 2024

Thanks for finding the associated ticket @svalexander , I've updated the title and pull request comment to refer to the ticket.

@aduth aduth requested a review from svalexander May 2, 2024 16:03
@aduth aduth merged commit c4cb898 into main May 2, 2024
@aduth aduth deleted the aduth-rm-ipp-outage-email branch May 2, 2024 16:10
@jmdembe jmdembe mentioned this pull request May 7, 2024
samathad2023 pushed a commit that referenced this pull request May 11, 2024
changelog: Internal, Code Quality, Remove unused code
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.

3 participants