discourse
/
discourse-assign
mirror of https://github.com/discourse/discourse-assign.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

DEV: Increase the buffer used to determine whether to send out the reminder email

This commit is contained in:
Nat 2023-08-15 13:27:06 +08:00
parent 3d5b9c5672
commit e6dbb36769
No known key found for this signature in database
GPG Key ID: C095C891379A319E
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/jobs/scheduled/enqueue_reminders.rb
View File

@ -20,7 +20,7 @@ module Jobs
Group.assign_allowed_groups.pluck(:id).join(",")
end
REMINDER_BUFFER_MINUTES = 10
REMINDER_BUFFER_MINUTES = 60
def user_ids
global_frequency = SiteSetting.remind_assigns_frequency

6
spec/jobs/scheduled/enqueue_reminders_spec.rb
View File

@ -52,16 +52,16 @@ RSpec.describe Jobs::EnqueueReminders do
user.custom_fields[
PendingAssignsReminder::REMINDERS_FREQUENCY
] = RemindAssignsFrequencySiteSettings::DAILY_MINUTES
user.custom_fields[PendingAssignsReminder::REMINDED_AT] = 1.days.ago
user.custom_fields[PendingAssignsReminder::REMINDED_AT] = 1.days.ago + 59.minutes
user.save
assign_multiple_tasks_to(user, assigned_on: 1.day.ago - 1.minute)
assign_multiple_tasks_to(user, assigned_on: 2.day.ago)
assert_reminders_enqueued(1)
end
it "does not enqueue a reminder if it's too soon" do
user.upsert_custom_fields(PendingAssignsReminder::REMINDED_AT => 1.days.ago)
user.upsert_custom_fields(PendingAssignsReminder::REMINDED_AT => 1.days.ago + 60.minutes)
assign_multiple_tasks_to(user)
assert_reminders_enqueued(0)