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

Fixed : Spacing for Simple APIs #26495

Merged
merged 8 commits into from
May 23, 2024
Merged

Conversation

akar016012
Copy link
Contributor

I have read the CONTRIBUTING.md file.

YES

What kind of change does this PR introduce?

Bug fix, feature, docs update, ...

Bug Fix

What is the current behavior?

image

What is the new behavior?

image

Additional context

added mt-16 to add a top margin so that the content is not cutoff

@akar016012 akar016012 requested a review from a team as a code owner May 17, 2024 19:23
Copy link

vercel bot commented May 17, 2024

@akar016012 is attempting to deploy a commit to the Supabase Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

vercel bot commented May 22, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
design-system ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 23, 2024 10:29am
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 23, 2024 10:29am
studio-staging ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 23, 2024 10:29am
zone-www-dot-com ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 23, 2024 10:29am
2 Ignored Deployments
Name Status Preview Comments Updated (UTC)
studio ⬜️ Ignored (Inspect) Visit Preview May 23, 2024 10:29am
studio-self-hosted ⬜️ Ignored (Inspect) Visit Preview May 23, 2024 10:29am

Copy link
Contributor

@fsansalvadore fsansalvadore left a comment

Choose a reason for hiding this comment

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

Thanks @akar016012, I left a few suggestions.

apps/www/pages/database/Database.tsx Outdated Show resolved Hide resolved
apps/www/pages/auth/Auth.tsx Outdated Show resolved Hide resolved
apps/www/pages/auth/Auth.tsx Outdated Show resolved Hide resolved
akar016012 and others added 2 commits May 22, 2024 16:45
Co-authored-by: Francesco Sansalvadore <f.sansalvadore@gmail.com>
@akar016012
Copy link
Contributor Author

Thanks @akar016012, I left a few suggestions.

Thank you, very helpful! 💯

@fsansalvadore fsansalvadore merged commit 1395f09 into supabase:master May 23, 2024
13 checks passed
@fsansalvadore
Copy link
Contributor

Merged, thanks @akar016012 🎉

@fsansalvadore
Copy link
Contributor

#26392

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.

None yet

2 participants