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.

Sign up for GitHub

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

Jump to bottom

Adding app metadata to setup and claims #15

Merged
merged 2 commits into from
Dec 3, 2023
Merged

Adding app metadata to setup and claims #15

merged 2 commits into from
Dec 3, 2023

Conversation

probablykabari
Copy link
Contributor

  • raw_app_meta_data column should have a value on creation
  • app_metadata is populated in the jwt.claims setting

@probablykabari
Adding app metadata to setup and claims
b8dcb14 
- raw_app_meta_data column should have a value on creation
- app_metadata is populated in the jwt.claims setting
@probablykabari probablykabari force-pushed the add-app-metadata-to-users branch from 162b4f7 to b8dcb14 Compare November 6, 2023 21:05
@tiniscule
Copy link
Contributor

This is great, thanks! Would you mind taking a quick look at that failing test? I can take a look in a bit if you don't have time.

@probablykabari
Fix authenticate-as test
a9939f2 
Update get_supabase_user function
@probablykabari
Copy link
Contributor Author

probablykabari commented Nov 27, 2023 •
edited
Loading

@tiniscule I've updated the test but tbh I'm not sure how that test was passing with checking for null before.

Additionally I updated another function to make sure the values would be correct when setting the claims.

@tiniscule tiniscule merged commit 75bd67e into usebasejump:main Dec 3, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@probablykabari @tiniscule