fix: caching of user and bucketed config #985

ajwootto · 2024-10-24T22:13:43Z

wrap the construction of the DVCPopulatedUser in a per-request cache so that it doesn't recreate from scratch on each call to useVariableValue
this also makes sure we're using the same timestamps for the user across the request
caching this means that caching the bucketed config result now works as intended
wrap the call to the SDK API in a cache as well, because the fetch call is being treated as "not cacheable" by Next for some reason

vercel · 2024-10-24T22:13:47Z

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
js-sdks-web-elements	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 24, 2024 10:29pm
js-sdks-with-provider	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 24, 2024 10:29pm

1 Skipped Deployment

Name	Status	Preview	Comments	Updated (UTC)
js-sdks-next-js-page-router	⬜️ Ignored (Inspect)			Oct 24, 2024 10:29pm

ajwootto changed the title ~~fix caching of user and bucketed config~~ fix: caching of user and bucketed config Oct 24, 2024

vercel bot deployed to Preview – js-sdks-web-elements October 24, 2024 22:15 View deployment

vercel bot deployed to Preview – js-sdks-with-provider October 24, 2024 22:15 View deployment

suthar26 approved these changes Oct 24, 2024

View reviewed changes

ajwootto enabled auto-merge (squash) October 24, 2024 22:19

fix caching of user and bucketed config

c812895

ajwootto force-pushed the fix-user-caching branch from 8b4ece8 to c812895 Compare October 24, 2024 22:26

vercel bot deployed to Preview – js-sdks-web-elements October 24, 2024 22:29 View deployment

vercel bot deployed to Preview – js-sdks-with-provider October 24, 2024 22:29 View deployment

ajwootto merged commit c7e36a0 into main Oct 24, 2024
11 checks passed

ajwootto deleted the fix-user-caching branch October 24, 2024 22:29

Provide feedback