Skip to content

migration + before_save hook to set recurrence.starts on absences and plage ouvertures

Vincent Agnano requested to merge fix/recurrence-starts into master

Created by: adipasquale

preliminary to #987

Montrose accepte une option starts optionnelle pour définir le début d'une récurrence. Pour l'instant nous ne la définissons jamais. Or, nos plages d'ouvertures et absences ont toujours une date de début.

L'absence du starts dans les recurrences ne semblait pas gênante jusqu'ici mais elle le devient lorsqu'on veut utiliser la méthode recurrence.include? dans la PR #987

dans cette PR :

  • migration qui sette le recurrence.starts sur toutes les POs et absences ou il est manquant
  • before_save qui s'assure que recurrence.starts = first_day lorsque recurrence n'est pas nil

J'ai testé la migration sur la db de prod, ca passe bien

Merge request reports