Skip to content

Cop idea: Prefer new syntax for sending delayed emails #7

@ydakuka

Description

@ydakuka

Describe the solution you'd like

https://github.com/sidekiq/sidekiq/wiki/Active-Job#action-mailer

# bad
UserMailer.delay_for(1.hour).welcome_email(@user.id)
UserMailer.delay_until(5.days.from_now).welcome_email(@user.id)

# good
UserMailer.welcome_email(@user).deliver_later(wait: 1.hour)
UserMailer.welcome_email(@user).deliver_later(wait_until: 5.days.from_now)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions