The instrumentation patch in T323160: NewImpact module: Instrumentation loads cached impact data when rendering the NewImpact module. Since we already access this data, I think it would make sense to export it to the client-side if it passes the "staleness" check (UserImpactHandler checks to see if page view data is older than 2 days).
- If it's stale, don't export anything and the useUserImpact.js would make an HTTP request to fetch data like it currently does.
- If it isn't stale, then useUserImpact.js wouldn't need to make any HTTP request, and we'd have a faster load time. A response with page view data is ~10kB, which isn't nothing, but is likely less expensive than an HTTP round-trip.