fix(mat-view-schedules): Fixes for scheduled materialized views

[phixMe]

Problem

Mostly trying to fix a production only weird update issue where the update isn't captured properly for the first sync interval.

Changes

• Clean up loading states
• Trying to fix bug

Does this work well for both Cloud and self-hosted?

Yes

How did you test this code?

Works the same locally, but hopefully fixes prod issue.

[github-actions]

Size Change: -5 B (0%)

Total Size: 1.21 MB

ℹ️ View Unchanged

Filename	Size	Change
frontend/dist/toolbar.js	1.21 MB	-5 B (0%)

_{compressed-size-action}

[greptile-apps]

PR Summary

This PR addresses issues with materialized view scheduling in PostHog by making the following key changes:

• Added loading state indicators to the InfoTab component in /frontend/src/scenes/data-warehouse/editor/OutputPaneTabs/InfoTab.tsx to improve UX during sync frequency updates
• Wrapped sync frequency updates in a transaction block in /posthog/warehouse/api/saved_query.py to ensure atomic operations
• Moved workflow sync operation outside the transaction block to prevent potential deadlocks
• Added proper handling of 'never' sync frequency by explicitly setting sync_frequency_interval to None
• Improved state management for the LemonSelect component to accurately reflect update status

The changes focus on fixing a production-specific issue where the first sync interval update wasn't being properly captured, while also enhancing the overall reliability of materialized view scheduling.

_{2 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile}