Skip to content

Conversation

@SaxonF
Copy link
Contributor

@SaxonF SaxonF commented Sep 10, 2025

image image

Adds GettingStarted row to the home page. This gives us a framework to start building more interactive steps with deeper integration with docs, dashboard, assistant.

@SaxonF SaxonF requested a review from a team as a code owner September 10, 2025 03:14
@vercel
Copy link

vercel bot commented Sep 10, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
design-system Ready Ready Preview Comment Sep 11, 2025 0:05am
docs Ready Ready Preview Comment Sep 11, 2025 0:05am
studio-self-hosted Ready Ready Preview Comment Sep 11, 2025 0:05am
studio-staging Ready Ready Preview Comment Sep 11, 2025 0:05am
ui-library Ready Ready Preview Comment Sep 11, 2025 0:05am
zone-www-dot-com Ready Ready Preview Comment Sep 11, 2025 0:05am
2 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
cms Ignored Ignored Sep 11, 2025 0:05am
studio Ignored Ignored Sep 11, 2025 0:05am

@supabase
Copy link

supabase bot commented Sep 10, 2025

This pull request has been ignored for the connected project xguihxuzqibwxjnimxev because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@SaxonF SaxonF requested a review from Copilot September 10, 2025 03:15
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a comprehensive "Getting Started" section to the home page that provides interactive guided workflows for new users. It implements a toggle between "no-code" and "code" workflows, each with customized step-by-step guidance and deep integration with documentation, dashboard features, and AI assistant.

  • Introduces a new GettingStartedSection with sortable positioning in the home layout
  • Implements code and no-code workflow paths with different step sequences
  • Integrates with existing project state to show completion status for various steps

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
Home.tsx Adds GettingStartedSection import and renders it conditionally within the sortable sections
GettingStartedSection.tsx Main component containing workflow logic, step definitions, and state management
GettingStarted.tsx Renders the step cards in a responsive grid layout with action buttons
FrameworkSelector.tsx Dropdown component for selecting development frameworks within getting started steps
Connect.tsx Updates Connect dialog to support URL parameters for pre-selecting tabs and frameworks

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@coveralls
Copy link

coveralls commented Sep 10, 2025

Coverage Status

coverage: 71.257%. remained the same
when pulling c573152 on feature/new-home-getting-started
into 8da4cbc on master.

@SaxonF SaxonF changed the title getting started row Home New: Getting Started Sep 10, 2025
Copy link
Contributor

@charislam charislam left a comment

Choose a reason for hiding this comment

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

Nice! Pushed a commit to fix a few minor nits + added some questions about the code-based workflow below, but otherwise it looks good, so approving to unblock!

() => [
{
key: 'install-cli',
status: 'incomplete',
Copy link
Contributor

Choose a reason for hiding this comment

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

Should there be a way to manually mark these steps (that can't be auto-checked) as complete? I personally would find it mildly infuriating to have them sitting around as incomplete perpetually 😄

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agree. Will create a follow-up PR adding this functionality

},
{
key: 'design-db',
status: tablesCount > 0 ? 'complete' : 'incomplete',
Copy link
Contributor

Choose a reason for hiding this comment

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

This might be sort of confusing given that it's mixing local and hosted flows. Since this is a local CLI Flow, 'design' should be considered complete once the user has a local migration file written (or pushed to their local DB), but this won't be complete until they have a table in their hosted DB.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Code first can be broken down further into local vs remote code first. For local, you run local env etc. For remote, you still use remote database but cli for pushing/pulling. Starting with remote flow first, then can maybe break down into local later?

Copy link
Contributor

Choose a reason for hiding this comment

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

Ahh fair point, makes sense!

],
},
{
key: 'add-rls-policies',
Copy link
Contributor

Choose a reason for hiding this comment

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

Again, this step might be confusing because of local/hosted mix.

We've been assuming that the user is working locally all along but 'Create a migration file' suddenly gets them to create a policy in hosted dashboard

@SaxonF SaxonF enabled auto-merge (squash) September 10, 2025 23:53
@SaxonF SaxonF merged commit 697709a into master Sep 11, 2025
19 of 20 checks passed
@SaxonF SaxonF deleted the feature/new-home-getting-started branch September 11, 2025 00:05
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.

3 participants