-
Notifications
You must be signed in to change notification settings - Fork 66
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[DBX-94127/94128] add 526 backup failure email job #18928
Conversation
Error: A file (or its parent directories) does not have a CODEOWNERS entry. Please update the .github/CODEOWNERS file and add the entry for the Offending file: app/sidekiq/form526_submission_failure_email_job.rb |
Error: A file (or its parent directories) does not have a CODEOWNERS entry. Please update the .github/CODEOWNERS file and add the entry for the Offending file: app/sidekiq/form526_submission_failure_email_job.rb |
Error: A file (or its parent directories) does not have a CODEOWNERS entry. Please update the .github/CODEOWNERS file and add the entry for the Offending file: app/sidekiq/form526_submission_failure_email_job.rb |
Error: A file (or its parent directories) does not have a CODEOWNERS entry. Please update the .github/CODEOWNERS file and add the entry for the Offending file: app/sidekiq/form526_submission_failure_email_job.rb |
Error: A file (or its parent directories) does not have a CODEOWNERS entry. Please update the .github/CODEOWNERS file and add the entry for the Offending file: app/sidekiq/form526_submission_failure_email_job.rb |
Error: A file (or its parent directories) does not have a CODEOWNERS entry. Please update the .github/CODEOWNERS file and add the entry for the Offending file: app/sidekiq/form526_submission_failure_email_job.rb |
Generated by 🚫 Danger |
ba7f6bf
to
957bdf7
Compare
Error: A file (or its parent directories) does not have a CODEOWNERS entry. Please update the .github/CODEOWNERS file and add the entry for the Offending file: app/sidekiq/form526_submission_failure_email_job.rb |
If so, just making a note that we may also need to add a silent_failure tag into wherever you are queuing this job, in the case that you fail to queue the job or throw an error when/before queuing it. correct, subsequent PR leverages this. We can discuss adding that extra tag in that PR : ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
small spelling nit-pick, but all good!
Backend-review-group approval confirmed. |
Summary
Related issue(s)
Testing done
Acceptance criteria