raggedstaff/openfoodnetwork
1
0
Fork 0
mirror of https://github.com/openfoodfoundation/openfoodnetwork synced 2026-03-07 03:01:33 +00:00
Code Issues Packages Projects Releases Wiki Activity

Manager invitation email is only for new users

Browse Source
This commit is contained in:
Cillian O'Ruanaidh
2026-02-28 14:30:19 +00:00
committed by Maikel Linke
parent f063e2e8c6
commit d57274fc4c
2 changed files with 7 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
app/forms/user_invitation.rb
View File

@@ -19,7 +19,10 @@ class UserInvitation
user = find_or_create_user!
enterprise.users << user
EnterpriseMailer.manager_invitation(enterprise, user).deliver_later
if user.previously_new_record?
EnterpriseMailer.manager_invitation(enterprise, user).deliver_later
end
end
private

8
spec/forms/user_invitation_spec.rb
View File

@@ -53,13 +53,11 @@ RSpec.describe UserInvitation do
end
context "inviting a existing user who isn't a user on the enterprise" do
it "adds the user to the enterprise and sends them an invitation email" do
it "adds the user to the enterprise" do
existing_user = create(:user)
user_invitation = UserInvitation.new(defaults.merge(email: existing_user.email))
expect do
user_invitation.save!
end.to have_enqueued_mail(EnterpriseMailer, :manager_invitation)
user_invitation = UserInvitation.new(defaults.merge(email: existing_user.email))
user_invitation.save!
expect(enterprise.users).to include(existing_user)
end