fix: remove basic mode for saved insights

[ndaba1]

Problem

Currently, saved insights are fetched in basic mode which is faster but contains a smaller payload. On the initial fetch, the insights won't contain the results key which is used when rendering the visualizations. Since the result is cached, when a user tries to swicth to Card layout on the product analytics tab, it uses the cached result which doesn't contain the results causing the empty state to be rendered.

On refresh, this issue doesn't exist since there's no initial cache state, and the dataNodeLogic response loader forces a refresh of each insight individually

Closes #39703

Changes

Removes basic mode for saved insights rendered on the Product analytics page

How did you test this code?

• Go to the product analytics page
• Switch to card layout
• The visualizations should be rendered correctly without having to refresh the page

[greptile-apps]

_{1 file reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}

[posthog-bot]

This PR hasn't seen activity in a week! Should it be merged, closed, or further worked on? If you want to keep it open, post a comment or remove the stale label – otherwise this will be closed in another week. If you want to permanentely keep it open, use the waiting label.

[ndaba1]

hey @clr182
Could you help get this reviewed

[posthog-bot]

This PR hasn't seen activity in a week! Should it be merged, closed, or further worked on? If you want to keep it open, post a comment or remove the stale label – otherwise this will be closed in another week. If you want to permanentely keep it open, use the waiting label.