fix(bulk messages): display previous bulk messages

This commit is contained in:
Eric Leroy-Terquem 2024-02-27 18:02:26 +01:00
parent 1f0e2030be
commit 6be57d9ea4
5 changed files with 7 additions and 4 deletions

View file

@ -258,7 +258,7 @@ module Instructeurs
def email_usagers
@procedure = procedure
@bulk_messages = BulkMessage.includes(:groupe_instructeurs).where(groupe_instructeurs: { procedure: procedure })
@bulk_messages = BulkMessage.where(procedure: procedure)
@bulk_message = current_instructeur.bulk_messages.build
@dossiers_without_groupe_count = procedure.dossiers.state_brouillon.for_groupe_instructeur(nil).count
end
@ -283,7 +283,8 @@ module Instructeurs
dossier_state: Dossier.states.fetch(:brouillon),
sent_at: Time.zone.now,
instructeur_id: current_instructeur.id,
groupe_instructeurs: GroupeInstructeur.for_dossiers(dossiers)
groupe_instructeurs: GroupeInstructeur.for_dossiers(dossiers),
procedure_id: @procedure.id
)
bulk_message.save!

View file

@ -33,5 +33,5 @@
.width-100
%h2.claimant
%span.email= message.instructeur.email
%span.date message envoyé à #{@dossiers_count} usagers le #{message.sent_at.strftime('%d/%m/%y à %H:%M')}
%span.date a envoyé ce message à #{@dossiers_without_groupe_count} usagers le #{message.sent_at.strftime('%d/%m/%y à %H:%M')}
%p= message.body

View file

@ -183,7 +183,7 @@ describe Administrateurs::GroupeInstructeursController, type: :controller do
let!(:gi_1_3) { create(:groupe_instructeur, label: 'groupe instructeur 3', procedure: procedure) }
let!(:dossier12) { create(:dossier, :en_construction, :with_individual, procedure: procedure, groupe_instructeur: gi_1_1) }
let!(:instructeur) { create(:instructeur) }
let!(:bulk_message) { BulkMessage.create(dossier_count: 2, dossier_state: "brouillon", body: "hello", sent_at: Time.zone.now, groupe_instructeurs: [gi_1_1, gi_1_3], instructeur: instructeur) }
let!(:bulk_message) { BulkMessage.create(dossier_count: 2, dossier_state: "en_construction", body: "hello", sent_at: Time.zone.now, groupe_instructeurs: [gi_1_1, gi_1_3], instructeur: instructeur, procedure: procedure) }
describe 'when the new group is a group of the procedure' do
before do

View file

@ -704,6 +704,7 @@ describe Instructeurs::ProceduresController, type: :controller do
expect(BulkMessage.count).to eq(1)
expect(bulk_message.body).to eq("avant\napres")
expect(bulk_message.groupe_instructeurs).to be_empty
expect(bulk_message.procedure_id).to eq(procedure.id)
end
it "creates a flash notice" do

View file

@ -6,5 +6,6 @@ FactoryBot.define do
sent_at { 1.day.ago }
instructeur { association :instructeur }
groupe_instructeurs { [association(:groupe_instructeur, strategy: :build)] }
procedure { association :procedure }
end
end