呼叫贡献:将DelayedJob呼叫移至ActiveJob

嘿伙计们,我们一直在努力逐步迁移 DelayedJobActiveJob 在DEV代码库中。

lightalloy头像 lightalloy在2019年6月12日发表评论

如#2497中所述,减少对…的依赖 DelayedJob 我们还需要移动通过调用创建的作业 delay 方法 ActiveJob

这是电话列表:

  • (X) Slackbot.ping
  • () MembershipMailer.delay.new_membership_subscription_email
  • () MembershipMailer.delay.subscription_update_confirm_email
  • () MembershipMailer.delay.subscription_cancellation_email
  • () user.follow
  • () reaction.create
  • () ArticleAnalyticsFetcher.new.update_analytics
  • () HtmlVariantSuccess.create
  • () HtmlVariantTrial.create
  • () message.send_push
  • () RssReader.new.fetch_user
  • ()几个电话 trigger_delayed_index

每个电话的操作:

  • 创建一个对应的 ActiveJob,指定队列名称
  • 在作业中调用所需的方法
  • 更换 delay 方法调用作业 perform_later 呼叫
  • 您可能需要使用修改现有测试 perform_enqueued_job 代替 run_background_jobs_immediately 帮手

记得传递记录 ids(如果需要)而不是ActiveRecord对象到作业以避免反序列化错误(#1621)

在GitHub上查看

看看这个问题。我们有很多合并拉取请求,可以作为一个指导来帮助您入门,例如……

lightalloy于2019年7月29日评论说

这是什么类型的公关? (选中所有适用的)

  • (x)重构

描述

  • 将send_mention_notification移至 ActiveJob
  • 使用了@copasetickid的pr并对其进行了改进

相关门票和文件

#1996年

在GitHub上查看

快乐的编码

资讯来源:由0x资讯编译自DEV,原文:https://dev.to/devteam/call-for-contributions-move-delayedjob-calls-to-activejob-5d7l ,版权归作者所有,未经许可,不得转载
你可能还喜欢