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

Updated example code for ownership owner_id in storage #28366

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

acetrondi
Copy link

Corrected the example code in the documentation for storage in supabase
Corrected the example code in the documentation to reflect that owner_id is a text string, not a UUID. This update prevents errors caused by incorrect usage of owner_id as a UUID in examples.

I have read the CONTRIBUTING.md file.
YES

What kind of change does this PR introduce?
Docs update

What is the current behavior?
The current behavior in the documentation incorrectly suggests using owner_id as a UUID, leading to potential errors when users attempt to apply this in practice.

What is the new behavior?
The documentation now correctly reflects that owner_id is a text string, providing accurate guidance for users.

Additional context
This update ensures that developers who reference the documentation will no longer encounter errors related to incorrect data types when using owner_id in their projects.

Corrected the example code in the documentation to reflect that `owner_id` is a text string, not a UUID. This update prevents errors caused by incorrect usage of `owner_id` as a UUID in examples.
Copy link

vercel bot commented Aug 4, 2024

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

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
design-system ⬜️ Ignored (Inspect) Visit Preview Aug 4, 2024 0:44am

Copy link

vercel bot commented Aug 4, 2024

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

A member of the Team first needs to authorize it.

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.

1 participant