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

[Doc] Fix Supabase API handler example in NextJS tutorial #10310

Conversation

Ariyn
Copy link
Contributor

@Ariyn Ariyn commented Oct 28, 2024

Official supabase document describes to add Authorization token. document

Problem

I could not update supabase rows after follow this nextjs integration example.

Turns out I need to add authorization token too, when RLS is set.

Solution

Describe the solution this PR implements

How To Test

Describe the steps required to test the changes

Additional Checks

  • The PR targets master for a bugfix, or next for a feature
  • The PR includes unit tests (if not possible, describe why)
    Document fix
  • The PR includes one or several stories (if not possible, describe why)
    Document fix
  • The documentation is up to date

Also, please make sure to read the contributing guidelines.

Official supabase document describes to add Authorization token.
[document](https://supabase.com/docs/guides/api/quickstart)
It's impossible to update rows without this token when rls is set.
Copy link
Contributor

@slax57 slax57 left a comment

Choose a reason for hiding this comment

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

Thank you for this contribution.
This PR doesn't need to target next, as it is not a new feature, but fixes an issue in the docs. Can you please rebase it to target the master branch instead?
Thank you.

@Ariyn Ariyn changed the base branch from next to master October 28, 2024 14:26
@Ariyn
Copy link
Contributor Author

Ariyn commented Oct 28, 2024

I changed. Thank you for kind introduce. :)

Copy link
Contributor

@slax57 slax57 left a comment

Choose a reason for hiding this comment

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

Thanks!

@slax57 slax57 added this to the 5.3.2 milestone Oct 28, 2024
@slax57 slax57 merged commit 716e58b into marmelab:master Oct 28, 2024
2 checks passed
@slax57 slax57 changed the title [DOC] Add missing parameter in NextJs integration example [Doc] Fix Supabase API handler example in NextJS tutorial Oct 28, 2024
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.

2 participants