Skip to content

RELEASE: OpenCRVS cache service dependency injection#93

Merged
Ethical-Ralph merged 1 commit intoproductionfrom
develop
Feb 2, 2026
Merged

RELEASE: OpenCRVS cache service dependency injection#93
Ethical-Ralph merged 1 commit intoproductionfrom
develop

Conversation

@phrogwrld
Copy link
Contributor

@phrogwrld phrogwrld commented Feb 2, 2026

Description

This PR merges the fix from develop into production to restore the missing cache service dependency in the OpenCRVS service constructor, which was preventing the production build from running.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

Changes Made

File Changes
src/opencrvs/opencrvs.service.ts Re-added cacheService dependency injection to the constructor

Root Cause

The cache service dependency was accidentally removed from the OpenCRVS service constructor, causing a missing dependency error that blocked the production deployment.

Testing

  • Manual tests completed
  • Added unit tests
  • Added e2e tests

Related Github Issue(s)/Trello Ticket(s)

Checklist

  • Code follows project style guidelines
  • Self-review completed
  • Documentation updated

@phrogwrld phrogwrld changed the title RELEASE Merge develop into production: conditional validation, OpenCRVS improvements, and email updates Feb 2, 2026
@phrogwrld phrogwrld changed the title Merge develop into production: conditional validation, OpenCRVS improvements, and email updates RELEASE: OpenCRVS bug fix Feb 2, 2026
@phrogwrld phrogwrld changed the title RELEASE: OpenCRVS bug fix fix: restore OpenCRVS cache service dependency injection Feb 2, 2026
@phrogwrld phrogwrld changed the title fix: restore OpenCRVS cache service dependency injection RELEASE: OpenCRVS cache service dependency injection Feb 2, 2026
@phrogwrld phrogwrld marked this pull request as ready for review February 2, 2026 13:08
@Ethical-Ralph Ethical-Ralph merged commit e9ca555 into production Feb 2, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants