Skip to content
This repository was archived by the owner on Oct 9, 2025. It is now read-only.

Conversation

@dev-737
Copy link
Member

@dev-737 dev-737 commented Oct 9, 2024

This commit replaces the use of the cacheClient module with the getRedis module, which provides a direct Redis client. This change was made to improve the maintainability and flexibility of the caching functionality in the application.

The key changes are:

  • Replace cacheClient with getRedis() calls in various files, including ConnectedListUtils.ts, deleteMessage.ts, CooldownService.ts, storeMessageData.ts, and cacheUtils.ts.
  • Remove the traverseCursor function from cacheUtils.ts, as it is no longer needed with the new Redis client.
  • Update the import paths to use the new #utils directory structure.

These changes will help to centralize the Redis client usage and make it easier to maintain and update the caching functionality in the future.

This commit replaces the use of the `cacheClient` module with the `getRedis` module, which provides a direct Redis client. This change was made to improve the maintainability and flexibility of the caching functionality in the application.

The key changes are:

- Replace `cacheClient` with `getRedis()` calls in various files, including `ConnectedListUtils.ts`, `deleteMessage.ts`, `CooldownService.ts`, `storeMessageData.ts`, and `cacheUtils.ts`.
- Remove the `traverseCursor` function from `cacheUtils.ts`, as it is no longer needed with the new Redis client.
- Update the import paths to use the new `#utils` directory structure.

These changes will help to centralize the Redis client usage and make it easier to maintain and update the caching functionality in the future.
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 9, 2024

@dev-737 dev-737 changed the title feat(cache): replace cacheClient with Redis client function feat(cache): replace cacheClient with getRedis() Oct 9, 2024
@dev-737 dev-737 merged commit 75d1254 into main Oct 9, 2024
4 checks passed
@dev-737 dev-737 deleted the Virtual-branch-1 branch October 9, 2024 07:45
Copy link
Member Author

@dev-737 dev-737 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

w

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants