Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PM-13764] Send Notification When Updating Collection Management Settings #5230

Merged
merged 2 commits into from
Jan 9, 2025

Merge branch 'main' into jmccannon/ac/pm-13764-update-collection-sett…

7ab957d
Select commit
Loading
Failed to load commit list.
Merged

[PM-13764] Send Notification When Updating Collection Management Settings #5230

Merge branch 'main' into jmccannon/ac/pm-13764-update-collection-sett…
7ab957d
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Jan 7, 2025 in 0s

0.00% of diff hit (target 43.72%)

View this Pull Request on Codecov

0.00% of diff hit (target 43.72%)

Annotations

Check warning on line 809 in src/Core/AdminConsole/Services/Implementations/OrganizationService.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Core/AdminConsole/Services/Implementations/OrganizationService.cs#L807-L809 

Added lines #L807 - L809 were not covered by tests

Check warning on line 64 in src/Core/Models/PushNotification.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Core/Models/PushNotification.cs#L62-L64 

Added lines #L62 - L64 were not covered by tests

Check warning on line 252 in src/Core/NotificationHub/NotificationHubPushNotificationService.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Core/NotificationHub/NotificationHubPushNotificationService.cs#L242-L252 

Added lines #L242 - L252 were not covered by tests

Check warning on line 243 in src/Core/Platform/Push/Services/AzureQueuePushNotificationService.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Core/Platform/Push/Services/AzureQueuePushNotificationService.cs#L237-L243 

Added lines #L237 - L243 were not covered by tests

Check warning on line 158 in src/Core/Platform/Push/Services/MultiServicePushNotificationService.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Core/Platform/Push/Services/MultiServicePushNotificationService.cs#L155-L158 

Added lines #L155 - L158 were not covered by tests

Check warning on line 97 in src/Core/Platform/Push/Services/NoopPushNotificationService.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Core/Platform/Push/Services/NoopPushNotificationService.cs#L97 

Added line #L97 was not covered by tests

Check warning on line 252 in src/Core/Platform/Push/Services/NotificationsApiPushNotificationService.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Core/Platform/Push/Services/NotificationsApiPushNotificationService.cs#L246-L252 

Added lines #L246 - L252 were not covered by tests

Check warning on line 279 in src/Core/Platform/Push/Services/RelayPushNotificationService.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Core/Platform/Push/Services/RelayPushNotificationService.cs#L269-L279 

Added lines #L269 - L279 were not covered by tests

Check warning on line 101 in src/Notifications/HubHelpers.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Notifications/HubHelpers.cs#L96-L101 

Added lines #L96 - L101 were not covered by tests
View more details on Codecov