starred/Cleanuparr-Cleanuparr
1
0
Fork 0
mirror of https://github.com/Cleanuparr/Cleanuparr.git synced 2026-02-04 12:21:33 -06:00
Code Issues 38 Packages Projects Releases 10 Wiki Activity

[PR #284] [MERGED] Rework notifications system #189

New Issue
Closed
opened 2025-10-09 16:59:57 -05:00 by giteasync · 0 comments
giteasync commented 2025-10-09 16:59:57 -05:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/Cleanuparr/Cleanuparr/pull/284
Author: @Flaminel
Created: 8/31/2025
Status: ✅ Merged
Merged: 9/2/2025
Merged by: @Flaminel

Base: main ← Head: rework_notifications

📝 Commits (10+)

📊 Changes

73 files changed (+5060 additions, -1225 deletions)

View changed files

📝 code/backend/Cleanuparr.Api/Controllers/ConfigurationController.cs (+535 -52)
📝 code/backend/Cleanuparr.Api/DependencyInjection/MainDI.cs (+5 -3)
📝 code/backend/Cleanuparr.Api/DependencyInjection/NotificationsDI.cs (+8 -10)
➕ code/backend/Cleanuparr.Api/Models/NotificationProviders/CreateAppriseProviderDto.cs (+10 -0)
➕ code/backend/Cleanuparr.Api/Models/NotificationProviders/CreateNotifiarrProviderDto.cs (+8 -0)
➕ code/backend/Cleanuparr.Api/Models/NotificationProviders/CreateNotificationProviderBaseDto.cs (+20 -0)
➕ code/backend/Cleanuparr.Api/Models/NotificationProviders/TestAppriseProviderDto.cs (+10 -0)
➕ code/backend/Cleanuparr.Api/Models/NotificationProviders/TestNotifiarrProviderDto.cs (+8 -0)
➕ code/backend/Cleanuparr.Api/Models/NotificationProviders/UpdateAppriseProviderDto.cs (+10 -0)
➕ code/backend/Cleanuparr.Api/Models/NotificationProviders/UpdateNotifiarrProviderDto.cs (+8 -0)
➕ code/backend/Cleanuparr.Domain/Enums/NotificationEventType.cs (+13 -0)
➕ code/backend/Cleanuparr.Domain/Enums/NotificationProviderType.cs (+7 -0)
📝 code/backend/Cleanuparr.Infrastructure/Events/EventPublisher.cs (+1 -1)
📝 code/backend/Cleanuparr.Infrastructure/Features/Notifications/Apprise/AppriseProvider.cs (+36 -71)
📝 code/backend/Cleanuparr.Infrastructure/Features/Notifications/Apprise/AppriseProxy.cs (+4 -3)
📝 code/backend/Cleanuparr.Infrastructure/Features/Notifications/Consumers/NotificationConsumer.cs (+63 -9)
➕ code/backend/Cleanuparr.Infrastructure/Features/Notifications/INotificationConfigurationService.cs (+15 -0)
➖ code/backend/Cleanuparr.Infrastructure/Features/Notifications/INotificationFactory.cs (+0 -18)
���� code/backend/Cleanuparr.Infrastructure/Features/Notifications/INotificationProvider.cs (+4 -20)
➕ code/backend/Cleanuparr.Infrastructure/Features/Notifications/INotificationProviderFactory.cs (+8 -0)

...and 53 more files

📄 Description

No description provided

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/Cleanuparr/Cleanuparr/pull/284 **Author:** [@Flaminel](https://github.com/Flaminel) **Created:** 8/31/2025 **Status:** ✅ Merged **Merged:** 9/2/2025 **Merged by:** [@Flaminel](https://github.com/Flaminel) **Base:** `main` ← **Head:** `rework_notifications` --- ### 📝 Commits (10+) - [`19eb16f`](https://github.com/Cleanuparr/Cleanuparr/commit/19eb16f1bca2b3938d15dffe72ad8fb1e03b2ca7) phase #1 - [`df7652d`](https://github.com/Cleanuparr/Cleanuparr/commit/df7652d1ada3db09caffea83d4bd8fb1803e5233) phase #2: refactored providers - [`6b406cf`](https://github.com/Cleanuparr/Cleanuparr/commit/6b406cf95711aa19e5fadf07a132bb02a60428f0) phase #3: API integration - [`a31aed6`](https://github.com/Cleanuparr/Cleanuparr/commit/a31aed67d2cf56335498207b5dd4c0cf08f58a48) phase #4: frontend integration - [`222fd42`](https://github.com/Cleanuparr/Cleanuparr/commit/222fd42ec12635c0a04ef4ab25c0dc6beb1220d5) fixed notification component - [`fdc2033`](https://github.com/Cleanuparr/Cleanuparr/commit/fdc203391befbe71711d7b50c5474f1a7eefb399) rework notification modals #1 - [`a9185be`](https://github.com/Cleanuparr/Cleanuparr/commit/a9185be2835432aa16b247b58d6f15bcfba5f0dd) rework notification modals #2 - [`6835e28`](https://github.com/Cleanuparr/Cleanuparr/commit/6835e28f93382ecb9e41e3d342c381c613867b62) fixed notification settings page layout - [`afad16a`](https://github.com/Cleanuparr/Cleanuparr/commit/afad16a5f2cb3b753b95afd90a6df1a5a3617b63) fixed notification provider components layout - [`b8d19dc`](https://github.com/Cleanuparr/Cleanuparr/commit/b8d19dc772849943527e3a62d851a86381ad9bda) fixed event fields duplication ### 📊 Changes **73 files changed** (+5060 additions, -1225 deletions) <details> <summary>View changed files</summary> 📝 `code/backend/Cleanuparr.Api/Controllers/ConfigurationController.cs` (+535 -52) 📝 `code/backend/Cleanuparr.Api/DependencyInjection/MainDI.cs` (+5 -3) 📝 `code/backend/Cleanuparr.Api/DependencyInjection/NotificationsDI.cs` (+8 -10) ➕ `code/backend/Cleanuparr.Api/Models/NotificationProviders/CreateAppriseProviderDto.cs` (+10 -0) ➕ `code/backend/Cleanuparr.Api/Models/NotificationProviders/CreateNotifiarrProviderDto.cs` (+8 -0) ➕ `code/backend/Cleanuparr.Api/Models/NotificationProviders/CreateNotificationProviderBaseDto.cs` (+20 -0) ➕ `code/backend/Cleanuparr.Api/Models/NotificationProviders/TestAppriseProviderDto.cs` (+10 -0) ➕ `code/backend/Cleanuparr.Api/Models/NotificationProviders/TestNotifiarrProviderDto.cs` (+8 -0) ➕ `code/backend/Cleanuparr.Api/Models/NotificationProviders/UpdateAppriseProviderDto.cs` (+10 -0) ➕ `code/backend/Cleanuparr.Api/Models/NotificationProviders/UpdateNotifiarrProviderDto.cs` (+8 -0) ➕ `code/backend/Cleanuparr.Domain/Enums/NotificationEventType.cs` (+13 -0) ➕ `code/backend/Cleanuparr.Domain/Enums/NotificationProviderType.cs` (+7 -0) 📝 `code/backend/Cleanuparr.Infrastructure/Events/EventPublisher.cs` (+1 -1) 📝 `code/backend/Cleanuparr.Infrastructure/Features/Notifications/Apprise/AppriseProvider.cs` (+36 -71) 📝 `code/backend/Cleanuparr.Infrastructure/Features/Notifications/Apprise/AppriseProxy.cs` (+4 -3) 📝 `code/backend/Cleanuparr.Infrastructure/Features/Notifications/Consumers/NotificationConsumer.cs` (+63 -9) ➕ `code/backend/Cleanuparr.Infrastructure/Features/Notifications/INotificationConfigurationService.cs` (+15 -0) ➖ `code/backend/Cleanuparr.Infrastructure/Features/Notifications/INotificationFactory.cs` (+0 -18) ���� `code/backend/Cleanuparr.Infrastructure/Features/Notifications/INotificationProvider.cs` (+4 -20) ➕ `code/backend/Cleanuparr.Infrastructure/Features/Notifications/INotificationProviderFactory.cs` (+8 -0) _...and 53 more files_ </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
giteasync added the
pull-request
 label 2025-10-09 16:59:57 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
enhancement
enhancement
planned
pull-request
Mirrored from GitHub Pull Request
question
No Label
pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
giteasync
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/Cleanuparr-Cleanuparr#189
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?