-
Notifications
You must be signed in to change notification settings - Fork 26.5k
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
Chore/update with supa #69336
base: canary
Are you sure you want to change the base?
Chore/update with supa #69336
Conversation
Allow CI Workflow Run
Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer |
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
|
@@ -1,6 +0,0 @@ | |||
import { clsx, type ClassValue } from "clsx"; |
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.
dupe, already in utils.ts
@@ -0,0 +1,4 @@ | |||
# Update these with your Supabase details from your project settings > API | |||
# https://app.supabase.com/project/_/settings/api |
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.
this was deleted from Jonny's branch
@@ -72,7 +73,9 @@ If you wish to just develop locally and not deploy to Vercel, [follow the steps | |||
|
|||
Both `NEXT_PUBLIC_SUPABASE_URL` and `NEXT_PUBLIC_SUPABASE_ANON_KEY` can be found in [your Supabase project's API settings](https://app.supabase.com/project/_/settings/api) | |||
|
|||
5. You can now run the Next.js local development server: | |||
5. This template comes with the default shadcn/ui style initialized. If you instead want the `New York` style, delete `components.json` and [re-install shadcn/ui](https://ui.shadcn.com/docs/installation/next) |
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.
since we initialize shadcn for them
@@ -12,7 +12,7 @@ export function FormMessage({ message }: { message: Message }) { | |||
</div> | |||
)} | |||
{"error" in message && ( | |||
<div className="text-destructive-foreground border-l-2 border-destructive-foreground px-4"> | |||
<div className="text-red-500 border-l-2 border-destructive-foreground px-4"> |
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.
colour wasn't working in light mode, so went with red
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.
weird, i'll check what's wrong.
shouddn't use hard coded colors
branched off @MildTomato's PR here:
#69308