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

2026-08-19T23:05:49+03:00

agrynco commented

2026-08-19 23:05:49 +03:00

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: #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: #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: #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: #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: #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: #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: #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: #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: #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: #13

Merge pull request 'feat: consume transactional email notifications' (#14 ) from development into main bb0de0f2c1

Reviewed-on: #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: #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.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: hrynco/hrynco-notification-service#16