Allow HighlightCache to refresh cache for file
Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.
The following discussion from !44629 (merged) should be addressed:
-
@patrickbajao started a discussion: (+1 comment) Can we open a follow-up issue and add a link here?
🙂 Also I'm wondering, why do we need to force re-caching?
one thing we need to keep in mind here, without doing anything fancy, enabling either of these approaches would only impact new diffs, and I worry we'll see an issue with the caching akin to what we spotted when lifting the diff size limit (IE - since we don't pass the diff contents on to the cache when the diff is
collapsed, we'll only have[]in the cache without doing some extra steps. Extra complexity in this project will involve figuring out what needs doing there, specifically.)
Additionally, when working with increasing the diff size limit for individual files, we saw a similar issue where we would decide a file could be "expanded" but the existing cache had [] results. (!43778 (merged)) In this case, we also could've used a "force refresh the cache" options.