Reputation: 5599
I want to schedule a task that sends a reminder to a user 1 week after they've signed up to my heroku app.
I'm imagining it would be something like:
task :send_reminders => :environment do
User.send_reminder
end
In my user model
def self.send_reminder
User.each do |user|
if Time.now == user.created_at + 7.days
ReminderMailer.delay.remind(user)
end
end
end
Is it a bad idea (takes up dyno time) to check every record's created_at value to work out if it's been a week?
Upvotes: 0
Views: 299