Skip to content
Snippets Groups Projects

Feature/send validation emails

Merged Karel van Klink requested to merge feature/send-validation-emails into develop
All threads resolved!

Send out notification emails for all tasks that have failed in the last hour. Add running this task at an hourly schedule

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Karel van Klink marked this merge request as draft

    marked this merge request as draft

  • added 1 commit

    • 788958fe - Make the schedule run daily instead of hourly, and send all failed tasks in one big email

    Compare with previous version

  • Karel van Klink resolved all threads

    resolved all threads

  • Karel van Klink marked this merge request as ready

    marked this merge request as ready

  • Karel van Klink added 8 commits

    added 8 commits

    • 0408be41 - 1 commit from branch develop
    • 078fe247 - Add mail recipient to oss params
    • 372040a5 - Move email recipients into mailer function
    • a66741e7 - Rename validate_products to validate_geant_products in a helper method
    • 8a4de8a4 - Add task to send out email notifications for failed validation workflows
    • 780d52f2 - Move conftest that disallows external HTTP calls to parent module
    • 12cb9f53 - Prevent unit tests from sending emails
    • 46c20765 - Make the schedule run daily instead of hourly, and send all failed tasks in one big email

    Compare with previous version

  • Karel van Klink added 3 commits

    added 3 commits

    • 5418d156 - Add task to send out email notifications for failed validation workflows
    • ae90aae5 - Move conftest that disallows external HTTP calls to parent module
    • 04353eb8 - Send one email per day, with all failed tasks

    Compare with previous version

  • added 1 commit

    • c8e4965d - Use ProcessStatus enum values

    Compare with previous version

  • added 1 commit

    • 984700ca - replace string concatenation with f-string interpolation

    Compare with previous version

  • Mohammad Torkashvand approved this merge request

    approved this merge request

  • Mohammad Torkashvand resolved all threads

    resolved all threads

  • Mohammad Torkashvand enabled an automatic merge when the pipeline for 984700ca succeeds

    enabled an automatic merge when the pipeline for 984700ca succeeds

  • Please register or sign in to reply
    Loading