flagsmith
flagsmith copied to clipboard
docs: Add .Net caching to docs
Thanks for submitting a PR! Please check the boxes below:
- [X] I have run
pre-committo check linting - [X] I have added information to
docs/if required so people know about the feature! - [X] I have filled in the "Changes" section below?
- [X] I have filled in the "How did you test this code" section below?
- [X] I have used a Conventional Commit title for this Pull Request
Changes
Add content about .Net caching to docs
How did you test this code?
Tested locally running docusaurus
The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| docs | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Apr 25, 2024 10:04am |
| flagsmith-frontend-preview | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Apr 25, 2024 10:04am |
| flagsmith-frontend-staging | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Apr 25, 2024 10:04am |
I actually don't think the .NET caching implementation is usable in it's current state since it doesn't implement any method for maintaining the size. This will likely cause a memory leak in any of our customer's applications that utilise the caching and have a large number of identities. See issue here.
I agree, we also should allow the user to set the TTL.
As per the conversation, this docs PR will be closed until the feature is complete.