Add dynamic column generation for UnifiedLogs table#37239
Merged
MildTomato merged 12 commits intomasterfrom Jul 17, 2025
Merged
Add dynamic column generation for UnifiedLogs table#37239MildTomato merged 12 commits intomasterfrom
MildTomato merged 12 commits intomasterfrom
Conversation
Eliminated the 'host' field from filter fields, schema definitions, and column configuration in UnifiedLogs. This streamlines the logs interface and removes unused or unnecessary data.
Replaces static column definitions with a dynamic generator that hides columns when all their values are empty. The UnifiedLogs table now adapts its columns based on the current data, improving readability and relevance.
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
2 Skipped Deployments
|
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
MildTomato
commented
Jul 17, 2025
Added explicit cell and header class names for column widths in the UnifiedLogs table to ensure consistent sizing and improve layout control.
The dynamic columns generator now returns both columns and their visibility state, allowing UnifiedLogs to merge dynamic visibility with existing column visibility. This improves control over which columns are shown based on the data.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR enables dynamic column generation for the Unified Logs table, improving maintainability and flexibility. Key changes:
Dynamic Columns
• Added generateDynamicColumns in Columns.tsx to build columns based on data.
• Columns like method, pathname, and event_message are hidden if empty.
• Replaced static UNIFIED_LOGS_COLUMNS with dynamic version using useMemo in UnifiedLogs.tsx.
Simplified Definitions
• Removed unused metadata and class names from columns.
• Standardized column sizing and sorting behavior.
Fallback
• Added static fallback using generateDynamicColumns([]) when no data exists.