Skip to content

Commit 4754ef8

Browse files
FrostyApeOneFrostyApeOne
authored andcommitted
Added contributors missing permissions
1 parent f052209 commit 4754ef8

File tree

2 files changed

+8
-1
lines changed

2 files changed

+8
-1
lines changed

src/DfE.ExternalApplications.Application/Applications/Commands/AddContributorCommandHandler.cs

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -174,6 +174,13 @@ private async Task<Result<UserDto>> HandleExistingContributor(
174174
new[] { AccessType.Read, AccessType.Write },
175175
dbUser.Id!,
176176
applicationId);
177+
userFactory.AddPermissionToUser(
178+
existingContributor,
179+
existingContributor.Email,
180+
ResourceType.Notifications,
181+
new[] { AccessType.Read, AccessType.Write },
182+
dbUser.Id!,
183+
applicationId);
177184

178185
await unitOfWork.CommitAsync(cancellationToken);
179186

src/DfE.ExternalApplications.Application/Applications/EventHandlers/ContributorAddedEventHandler.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ protected override async Task HandleEvent(ContributorAddedEvent notification, Ca
4444

4545
userFactory.AddPermissionToUser(
4646
notification.Contributor,
47-
notification.ApplicationId.Value.ToString(),
47+
notification.Contributor.Email,
4848
ResourceType.Notifications,
4949
new[] { AccessType.Read, AccessType.Write },
5050
notification.AddedBy,

0 commit comments

Comments
 (0)