Skip to content

LG-8739: Change queue used for USPS proofing job emails to prevent queue alarm#7738

Merged
NavaTim merged 2 commits intomainfrom
tbradley/lg-8739-change-usps-job-mail-queue
Jan 31, 2023
Merged

LG-8739: Change queue used for USPS proofing job emails to prevent queue alarm#7738
NavaTim merged 2 commits intomainfrom
tbradley/lg-8739-change-usps-job-mail-queue

Conversation

@NavaTim
Copy link
Contributor

@NavaTim NavaTim commented Jan 31, 2023

🎫 Ticket

🛠 Summary of changes

📜 Testing Plan

Provide a checklist of steps to confirm the changes.

  • Ensure that a delay is configured via in_person_results_delay_in_hours exceeding 10 seconds
  • Deploy both IDP and infra changes
  • Have a user complete the proofing process
  • Verify that success/failed/fraud email was delivered according to the configured delay
  • Verify that the alarm for job queue time was not triggered

@NavaTim NavaTim requested review from a team and racingspider January 31, 2023 21:28
@NavaTim NavaTim changed the title LG-8737: Change queue used for USPS proofing job emails to prevent queue alarm LG-8739: Change queue used for USPS proofing job emails to prevent queue alarm Jan 31, 2023
@NavaTim NavaTim force-pushed the tbradley/lg-8739-change-usps-job-mail-queue branch from 81d3658 to fc6ffac Compare January 31, 2023 21:29
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

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

LGTM

params: { user: user, email_address: user.email_addresses.first },
args: [{ enrollment: pending_enrollment }],
).at(Time.zone.now + 1.hour)
).at(Time.zone.now + 1.hour).on_queue(:intentionally_delayed)
Copy link
Contributor

Choose a reason for hiding this comment

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

ooo what a neat matcher, TIL

@NavaTim NavaTim merged commit 0f45b2b into main Jan 31, 2023
@NavaTim NavaTim deleted the tbradley/lg-8739-change-usps-job-mail-queue branch January 31, 2023 22:03
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