Accept glob pattern or RegExp in trigger() about swr HOT 6 CLOSED

I ended up with this by the way:

function revalidate(matcher: string | RegExp) {
  cache
    .keys()
    .filter(key => key.match(matcher))
    .forEach(key => mutate(key));
}

from swr.

I just saw that #78 is kinda related. Feel free to close if it's the same thing!

from swr.

Yeah accepting glob pattern or RegExp is a good idea, but it will be unnecessary to add it as an option after giving you the full control of the cache.

from swr.

I think this could now be solved with the exposed cache, get all cache keys, filter based on a glob and then call mutate against them without data to revalidate them or call cache.delete to delete them altogether

from swr.

I think 'trigger' is s feature not directly related to the cache. As a library user I would like to be able to call 'trigger' without necessarily knowing how the cache works internally.

I like your take @JulianG and this is also my opinion about why I personally don't want cache APIs to be documented for now. Because ideally they should be covered by some other high level APIs or plugins. It's dangerous for end users to touch the cache.

(I'd close this PR, it's kinda unrelated to the topic)

from swr.

Hello!

I think I see the point in not bloating the library with features that can be done by combining the existing feature set, but

I don't think we can rely on the exposed cache for this: (let me know if I'm wrong) (I was wrong)

You may want to only revalidate "active" or visible queries only. For example, if I update "/users/42" and now I want to revalidate also "/users/", "/users/42" and any other "visible" users but not every user in the cache. Assuming other users would still be revalidated when/if they're requested again.

Also it may not be desirable to rely on the cache for this: (this is just opinion)

I think 'trigger' is s feature not directly related to the cache. As a library user I would like to be able to call 'trigger' without necessarily knowing how the cache works internally.

from swr.