Skip to content

Feat/update metadata page - #183

Merged
YanivZalach merged 8 commits into
masterfrom
feat/update_metadata_page
Sep 24, 2026
Merged

YanivZalach merged 8 commits into
masterfrom
feat/update_metadata_page

Conversation

@YanivZalach

Copy link
Copy Markdown
Owner

No description provided.

Move specs to dedicated feature folder
Move pages to top-level folder
Rename MetadataStructured to SpecHistory
Comment on lines +17 to +20
Click the **Schema**, **Spec**, or **Order** link to open that definition in the Specs panel. Use **Full / Diff** there to inspect changes. The schema section also reports earlier schemas available in this metadata file.

Each partition and sort row resolves the source column name against the current schema and shows its source ID. Sort fields are numbered in priority order. The sort order is a write preference, and it does not guarantee file or query result order.

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not needed here.

Comment thread frontend/src/features/metadata/components/MetadataPartitioning.tsx Outdated
Comment thread frontend/src/features/metadata/components/MetadataSortOrder.tsx Outdated
Comment on lines +25 to +26
// The committed time is a rendered string, so it is shown without a copy
// action rather than offering a value that is not the stored timestamp.

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need for the comment.

@@ -0,0 +1,7 @@
// Expandable sections sit directly on top of each other, so the summary stays

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is that the style we support in the rest of the project? dont we have a single style class for all?

Comment thread frontend/src/features/schema/schemaDiffTypes.ts
@YanivZalach
YanivZalach force-pushed the feat/update_metadata_page branch 3 times, most recently from ece32d6 to f7cdae8 Compare September 24, 2026 13:38
@YanivZalach
YanivZalach force-pushed the feat/update_metadata_page branch from f7cdae8 to 4b84216 Compare September 24, 2026 18:57
@YanivZalach
YanivZalach merged commit 43d872d into master Sep 24, 2026
4 checks passed
@YanivZalach
YanivZalach deleted the feat/update_metadata_page branch September 24, 2026 23:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant