Notification service types - #321
Open
haydentho wants to merge 4 commits into
Open
Conversation
- Add shared CacheService (get/set/delete/deleteMany/exists) backed by the existing Redis client, with cache-aside pattern and fail-soft error handling (logs and returns safe defaults instead of throwing, so a Redis outage never takes the API down) - Extend Redis client with connection/error logging, checkRedisHealth(), and closeRedisConnection() for graceful shutdown - Add Jest + ts-jest as shared backend devDependencies, with a single root jest.config.js resolving @phoenix/* path aliases so any service can add test files without extra setup - Exclude *.test.ts/*.spec.ts from all service tsconfig builds so test code never leaks into production dist output - Add 12 unit tests for CacheService covering cache hit/miss, malformed JSON eviction, and Redis-down failure handling (all passing)
Documents the required Member 5 analysis output: cache-candidate list and priority ranking, recommended caching pattern, Redis connection lifecycle, configuration requirements, and proposed initial endpoint (getEventStatuses) for Phase 1 group sign-off, per the task doc's Individual Completion Evidence requirements.
…ce-Titiksha-backend S2-T3: Add Redis CacheService foundation with Jest test setup
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.