def send_membership_reminders!
today = Time.now.to_date
+ reminders = NosOignons::Reminder.all.sort_by(&:days)
NosOignons::Member.all.select(&:up_to_date?).each do |member|
- next if member.membership_fee_paid_on.year == today.year
- reminders = NosOignons::Reminder.all.sort_by(&:days)
- anniversary = Time.new(today.year, member.joined_on.month,
+ next if member.membership_fee_paid_on.year == (today + reminders.last.days).year
+ anniversary = Time.new(today.next_month.year, member.joined_on.month,
member.joined_on.day).to_date
next if member.membership_fee_paid_on >= anniversary - reminders.last.days
reminders.each do |reminder|