Simon Hamp
Simon Hamp
Ok, so it trashes the test suite. No problem, I will dig further into this.
I've been flipping between this code and the currently-released code and I'm more convinced now that the `cachedFlattenedPages` is the main culprit here. Just by sharing that cache for a...
Thanks for trying it out and sharing that data @jelleroorda Can I check, is that the Time To First Byte value that you've captured in your table? Or something else.
Also, when you say "on the staging server loads in ~1.3s", is that with or without this patch? It does depend on how many `{{ nav }}` tags are used...
TIL! Thanks @duncanmcclean
> The 1.3s is on the staging server without the patch. That's pretty speedy already @jelleroorda, are you doing any kind of caching around any of those navs? Or are...
Awesome! So you'll still see a speed up if some of those navs aren't cached. My next PR (Coming Soon™️) takes this even further with caching of its own baked...
> Not sure how I still measures a ~6% speed improvement in my local environment though, since that also has the cache tags enabled. Docker?
Different Stache/Cache drivers?
So, I've ended up adding the caching here. It was simpler than I thought... because the Tree Blink cache key is a hash of the actual tree, the cache is...