fix: Add Cache Wrapper helper to avoid dataset request duplication

[agatapst]

SUMMARY

The problem was that datasource and field are requested from API every time (instead of keeping them cached when they have been fetched once). The aim of the PR was to limit the API requests using Cache Wrapper helper.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Before:

After:

TEST PLAN

Verify manually.
Create new filter or edit existing one.
Choose datasource and field and check the network if it works the same as presented on the gif.

ADDITIONAL INFORMATION

• Has associated issue:
• Changes UI
• Requires DB Migration.
• Confirm DB Migration upgrade and downgrade tested.
• Introduces new feature or API
• Removes existing feature or API

cc @villebro

[villebro]

This looks good, but it would be great to get a few tests for cacheWrapper, as I'm sure once this goes in others will use it, too. Something like making sure fn is called once with args when the key isn't found, and subsequent calls return the correct value without fn being called, even when args gets parameters that aren't covered by keyFn.

[villebro]

Perfect, this is above and beyond what I had hoped for 🙏 👏