Félix Malfait
Félix Malfait
Oh man I'm sorry. There was an ongoing PR for this which I just merged. I apologize, I should have noticed and linked the original issue :( Let me know...
Thanks! Feel free to look at issues tagged "good first issues" (easier) or "good for experienced contributors" (hard). I'll try to do a cleanup of those issues today
@Jonathanlemon of course, that'd be very helpful if you can solve this. Got this feedback twice from users. Thanks!
@Jonathanlemon did you have time to look yet? Thanks!
OK no problem @Jonathanlemon thanks for trying!
@capaj Tbh I didn't really think about it in details while writing the issue. The issue with in-memory is that it would be hard to invalidate it. We have a...
@capaj No I meant that we already have an abstraction layer called "Cache Storage . Service" you can find the file in the project. Env var is `REDIS_HOST`
https://github.com/twentyhq/twenty/blob/main/packages/twenty-server/src/engine/integrations/cache-storage/cache-storage.service.ts https://github.com/twentyhq/twenty/blob/main/packages/twenty-server/src/engine/integrations/cache-storage/cache-storage.module.ts https://docs.nestjs.com/techniques/caching The file you're looking at is not up-to-date btw (not on main branch anymore)
What we could do eventually is put an additional layer for the config so that the first time we ask for a config flag it's retrieved from redis but all...
Thanks @saadfrhan ; @capaj are you working on it already?