New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Existing multicaching support not exposed in types #174
Comments
Would you like to create a PR for this issue? |
I'm open to doing so. I'm not sure when I can invest the time just yet, got a few critical projects that need to see completion in the coming month. |
hello there 👋🏻 |
Hi @Djunnni, There exists a way to set up multi-caching. I saw it in the test case: cache-manager/tests/src/multi-store/multi-store.module.ts Lines 8 to 21 in 2a82f89
But I think it is very hard to read CacheModuleOptions to know there is an existing way config like that. cache-manager/lib/interfaces/cache-module.interface.ts Lines 4 to 15 in 2a82f89
So I suggest changing the type of CacheModuleOptions so everyone can easily set up when using the lib. |
Is there an existing issue for this?
Current behavior
Technically, the code supports multicaching, as written here: https://github.com/nestjs/cache-manager/blob/master/lib/cache.providers.ts#L53-L59
However, the types set on the register function do not indicate any sort of array input support.
There's also no mention of multicaching support in the documentation
Minimum reproduction code
https://github.com/nestjs/cache-manager/blob/master/lib/cache.providers.ts#L53-L59
Steps to reproduce
No response
Expected behavior
Multicaching should be supported by the
register
andregisterAsync
methods.Package version
2.0.1
NestJS version
10.0.5
Node.js version
No response
In which operating systems have you tested?
Other
No response
The text was updated successfully, but these errors were encountered: