hrynco/hrynco-notification-service
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

merge: IT-628 RabbitMQ worker, contracts, usage UI in channels screen #16

Closed
agrynco wants to merge 51 commits from feature/it-1115-html-template-escaping into IT-628
pull from: feature/it-1115-html-template-escaping
merge into: hrynco:IT-628
Conversation 0 Commits 51 Files Changed 114
+3216 -764
agrynco commented 2026-08-19 23:05:49 +03:00
Owner
Copy Link
Copy Source
No description provided.
agrynco added 51 commits 2026-08-19 23:05:50 +03:00
merge: IT-628 RabbitMQ worker, contracts, usage UI in channels screen 4a431ec6c6
fix: move ManagePackageVersionsCentrally to Directory.Packages.props 5003ab8764 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
merge: development -> main 18f7981ccc
chore: add NuGet metadata to Contracts project 74211f0a4a 
- Add PackageId, Authors, Description, PackageTags, RepositoryUrl
- Matches metadata pattern from HrynCo.Common and HrynCo.RabbitMq

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Merge branch 'development' c303514414
feat: add production docker-compose with hrynco-services network ae119d1a3d 
- Base compose: explicit internal network, named volumes with VOLUME_PREFIX
- docker-compose.prod.yml: production images, ports, restart policies, hrynco-services external network on rabbitmq
- docker-compose.Development.yml: cleaned up orphan volumes, named dev volumes

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Merge branch 'development' a6f9a0a530
chore: update package versions and formatting in Directory.Packages.props c88511ce3b 
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Merge branch 'development' 8dab3c0dc0
fix: wait for postgres healthcheck before running migrator 166b1a6103 
- Add pg_isready healthcheck to db service (5s interval, 10 retries)
- Change migrator depends_on condition: service_started -> service_healthy
- Prevents migrator connection failure on fresh postgres startup

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Merge branch 'development' 09c3985fad
fix: add internal network to migrator, api, worker services c5528b253d 
- migrator, api, worker were missing 'networks: - internal'
- db and rabbitmq are only on internal network, so services couldn't reach them
- also changed api depends_on db condition to service_healthy

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Merge branch 'development' 859ae0b50d
fix: add missing FK migration for EmailChannelUsage -> EmailChannel d71c3513a5 
- EF model had a pending HasOne/WithMany relationship not in migrations
- Adds FK_email_channel_usage_email_channels_provider_id with cascade delete

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Merge branch 'development' 9a0aaf629b
fix: rename api service to web in all docker-compose files 3e1cc696c1 
- Aligns compose service name with the image name (hrynco.notification-service.web)
- Rename API_PORT env var to WEB_PORT for consistency

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Merge branch 'development' b07cd06477
feat: add test button to create channel form using ad-hoc smtp test endpoint 6302a07178 
- Add TestSmtpCommand and TestSmtpHandler for ad-hoc smtp testing without saving
- Add POST /admin/channels/test-smtp endpoint accepting raw smtp settings
- Show Test button on both Create and Edit forms
- Test reads current form values so channel can be tested before saving
Merge branch 'development' 5c7b5f7b10
chore: add Application property to Serilog for log filtering 49982fc27f 
- Worker logs tagged as 'hrynco-notification-service-worker'
- Web logs tagged as 'hrynco-notification-service-web'

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Merge pull request 'release: development -> main' (#1) from development into main 9c2edd4712
refactor: replace local dal abstractions with hrynco-ef packages ee4c988a0d 
Remove duplicate IEntity, Entity, ITransaction, IUnitOfWork, EfRepository,
EfUnitOfWork, EfTransactionAdapter — now consumed from HrynCo.DAL.Abstract
and HrynCo.DAL.EF packages (1.0.1).

Ref: IT-0
Merge pull request 'refactor: replace local DAL abstractions with hrynco-ef packages' (#2) from use-hrynco-ef-packages into development 334bf2a567
Merge pull request 'refactor: replace local DAL abstractions with hrynco-ef packages' (#3) from development into main 637af06a9c
chore: update package versions and refactor TransactionBehavior 0d1aa4f6be 
- Upgrade EF Core packages to 10.0.x and HrynCo.DAL.Abstract to 1.0.10
- Refactor TransactionBehavior to simplify transaction handling logic
feat: add real-time email template preview with variable interpolation 9490718c04 
- Introduce preview panel to edit form for rendering subject, HTML, and text with sample values.
- Add support for real-time updates using JavaScript.
- Include supporting styles for preview panel in admin CSS.
- Add optional `Scripts` section rendering to `_EditorLayout`.
- Create `.env.Development` for better development environment configuration.
Merge pull request 'chore: update package versions and refactor TransactionBehavior' (#4) from development into main 7b77062aa7 
Reviewed-on: https://gitea.grynco.com.ua/hrynco/hrynco-notification-service/pulls/4
refactor: use AsNoTracking for email template queries to improve performance b4d8497ea7
Merge pull request 'refactor: use AsNoTracking for email template queries to improve performance' (#5) from development into main f3966705ad 
Reviewed-on: https://gitea.grynco.com.ua/hrynco/hrynco-notification-service/pulls/5
refactor: replace internal UnitOfWork with NotificationUnitOfWork and NotificationBaseRepository 50828d23ec 
- Consolidate unit of work implementation with NotificationUnitOfWork.
- Refactor repositories to use NotificationBaseRepository for consistency.
- Simplify request handlers by removing IUnitOfWork dependency.
- Update related tests and service registration.
refactor: update admin CSS for improved layout and readability c18f0b7fb1 
- Switch `.template-preview-variables` to grid layout for better responsiveness.
- Adjust form label font size and spacing for consistency.
- Enhance styles of `.form-control-sm` for improved usability.
Merge pull request 'refactor: replace internal UnitOfWork with NotificationUnitOfWork and NotificationBaseRepository' (#6) from development into main dfa097eb43 
Reviewed-on: https://gitea.grynco.com.ua/hrynco/hrynco-notification-service/pulls/6
refactor: improve UpdateAsync in EmailTemplateRepository for better entity handling 0861e18cec 
- Add null check and fetch entity before updating fields.
- Replace the direct property assignment with detailed updates.
- Ensure changes are saved using SaveChangesAsync.
Merge pull request 'refactor: improve UpdateAsync in EmailTemplateRepository for better entity handling' (#7) from development into main ab042c7617 
Reviewed-on: https://gitea.grynco.com.ua/hrynco/hrynco-notification-service/pulls/7
refactor: modularize email processing logic and improve service structure 25fb48ccf0 
- Extract email template handling, rendering, and sending code into `Worker.Services` project.
- Introduce `EmailTemplateService`, `EmailTemplateRenderingService`, and `SendEmailService`.
- Simplify consumer logic by delegating to scoped services.
- Update project dependencies and package references accordingly.
Merge pull request 'refactor: modularize email processing logic and improve service structure' (#8) from development into main 9589095760 
Reviewed-on: https://gitea.grynco.com.ua/hrynco/hrynco-notification-service/pulls/8
refactor: simplify Program.cs and clean up NuGet.Config 285cc6abb7
Merge pull request 'refactor: simplify Program.cs and clean up NuGet.Config' (#9) from development into main 3bd952ef9e 
Reviewed-on: https://gitea.grynco.com.ua/hrynco/hrynco-notification-service/pulls/9
refactor: add ordering by ServiceName and Key in GetAllAsync method 3381fcc2f8
Merge pull request 'refactor: add ordering by ServiceName and Key in GetAllAsync method' (#10) from development into main e7d3953747 
Reviewed-on: https://gitea.grynco.com.ua/hrynco/hrynco-notification-service/pulls/10
feat: add filtering by ServiceName and Key in email templates query and UI 07f536938f
Merge pull request 'feat: add filtering by ServiceName and Key in email templates query and UI' (#11) from development into main 94f0d45aaf 
Reviewed-on: https://gitea.grynco.com.ua/hrynco/hrynco-notification-service/pulls/11
feat: add persistence and restoration of email template filters in UI and backend cc3857a409
Merge pull request 'feat: add persistence and restoration of email template filters in UI and backend' (#12) from development into main b8435ac07b 
Reviewed-on: https://gitea.grynco.com.ua/hrynco/hrynco-notification-service/pulls/12
feat: consume transactional email notifications 2757869176 
Add contract validation, SMTP delivery results, terminal failure context, neutral development seeding, and local Docker setup.

Ref: IT-1033
docs: document repository git workflow 9d6b0717e6 
Record the main-based branch and delivery rules for future changes.

Ref: IT-1033
Merge pull request 'Merge pull request 'chore: update package versions and refactor TransactionBehavior' (#4) from development into main' (#13) from IT-1033 into development d5a2e538f0 
Reviewed-on: https://gitea.grynco.com.ua/hrynco/hrynco-notification-service/pulls/13
Merge pull request 'feat: consume transactional email notifications' (#14) from development into main bb0de0f2c1 
Reviewed-on: https://gitea.grynco.com.ua/hrynco/hrynco-notification-service/pulls/14
fix: correct notification template filter links 1ceb71b9a2 
Use explicit Razor interpolation for create and edit URLs and cover the rendered link syntax with a regression test.

Ref: IT-1039
Merge pull request 'fix: correct notification template filter links' (#15) from bugfix/it-1039-template-filter-links into main 01c622038d 
Reviewed-on: https://gitea.grynco.com.ua/hrynco/hrynco-notification-service/pulls/15
fix: escape html notification variables 50033a5bd4 
Encode template variables in HTML bodies while preserving text and subjects.

Ref: IT-1115
agrynco closed this pull request 2026-08-19 23:07:24 +03:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
agrynco (Anatolii Hrynchuk)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: hrynco/hrynco-notification-service#16
Delete Branch "feature/it-1115-html-template-escaping"

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?