-
Notifications
You must be signed in to change notification settings - Fork 23
Tim/eng 1189 create templates experience page on new docs #2963
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
base: main
Are you sure you want to change the base?
Tim/eng 1189 create templates experience page on new docs #2963
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
|
||
| ```bash filename="Terminal" copy | ||
| curl -X POST 'https://your-host/workflows/data-processing-<hash>/terminate' \ | ||
| -H 'Authorization: Bearer <plain_token_from_output>'' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
code sizeing new batch data insertion versioning & migration next batch next batch fix mdx-rendered and adding next batch of files adding api files fixed some language rendering and added more pages added workflows files added other workflow files streaming files added additional frameworks added new batch added top level files data modeling all files migrated initial changelog addressing cursor comments fix tabs added a bit of padding to the nav added sonner for language change fix TOC fix the build add ts ignore to mux player removed top nav fix TOC challenges added templates to new docs split server & client components moved to static api route instead
cca8415 to
05a2af0
Compare
| WHERE {where_clause} | ||
| LIMIT {limit} | ||
| OFFSET {offset} | ||
| """ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Query parameters: scope and definition.
The f-string query uses {limit} and {offset} which reference undefined variables. These should use parameter placeholders like the WHERE clause does, not f-string interpolation. The variables limit and offset don't exist in scope - only params.limit and params.page are available. The query should be LIMIT {limit} and OFFSET {offset} as ClickHouse parameter placeholders (without f-string interpolation), relying on the params_dict that already contains these values.
Note
Introduces a templates/apps explorer backed by a new API, adds rich MDX components (changelog, file tree, video, templates grid), improves code rendering/TOC/nav, and removes the old migration script.
TemplatesGridServer,template-grid,template-card) to browse templates/apps with search and filters.src/app/api/templates/route.tsserves items fromlib/templates.lib/templates.ts(readstemplates/*/template.config.toml, infers metadata) andlib/template-types.ts.mdx-rendererexposes many new components:ExportRequirement,MuxVideo,FileTree(with subcomponents), changelog components (ReleaseHighlights,Added, etc.),TemplatesGridServer, tabs/badge/icons,SourceCodeLink.export-requirement.tsx,mux-video.tsx,file-tree.tsx,changelog-category.tsx.components/ui/sonnerand mount<Toaster>inlayout.tsx; language tabs prompt to set default.fastify,koa,raw-nodejs) and null-safegetSectionFromPathname.scripts/migrate-content.ts.@iarna/toml,@mux/mux-player-react, andsonner.Written by Cursor Bugbot for commit 05a2af0. This will update automatically on new commits. Configure here.