Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat: Use source list component to display list of models #3380

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

mykalmax
Copy link
Member

No description provided.

@mykalmax mykalmax force-pushed the feat-shared-docs-components branch 3 times, most recently from f6375ee to d329302 Compare November 25, 2024 16:50
@mykalmax mykalmax marked this pull request as ready for review November 25, 2024 16:51
@mykalmax mykalmax force-pushed the feat-shared-docs-components branch from 8962171 to 6f5d918 Compare November 26, 2024 22:14
@mykalmax mykalmax force-pushed the feat-shared-docs-components branch 2 times, most recently from d1ed711 to a2ff26b Compare December 6, 2024 18:32
@RN-nguyen
Copy link

Looks like format in model details was missing columns and Max has since updated and fixed.

Verified catalog in sqlmesh, CLL, model search.

Looks good to me.

@mykalmax mykalmax force-pushed the feat-shared-docs-components branch 2 times, most recently from 20ca518 to 3239d7c Compare December 20, 2024 19:00
@mykalmax mykalmax force-pushed the feat-shared-docs-components branch from c9175e3 to 7da46c3 Compare January 2, 2025 17:42
@SQLMesh
Copy link

SQLMesh commented Jan 2, 2025

Verified fix for cron bug.

Regression test over data catalog, model descriptions, CLL, search, and plan.

@RN-nguyen
Copy link

Sorry above SQLMesh user was me^. Not sure what happened there, but have logged out and into my account since.

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.

4 participants