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

Feat: Extend the fields for users in the Auth0 FDW #265

Merged
merged 2 commits into from
May 23, 2024

Conversation

romaingrx
Copy link
Contributor

What kind of change does this PR introduce?

This PR extends the Auth0User struct to include all fields that the Auth0 endpoint may send (see Auth0 API Documentation).

What is the current behavior?

N/A

What is the new behavior?

The Auth0User struct now includes all possible fields, allowing us to use these fields in a foreign table and apply quals.

Additional context

N/A

@burmecia
Copy link
Member

Thanks for the PR! That's excellent but ere are some Clippy complaints, could you resolve it?

@romaingrx
Copy link
Contributor Author

romaingrx commented May 21, 2024

Hello, it should be all good now. Sorry, totally forgot to run clippy and fmt locally on the 2 PRs.

@burmecia burmecia added the auth0 Auth0 related issues label May 23, 2024
Copy link
Member

@burmecia burmecia left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@burmecia burmecia merged commit a85935b into supabase:main May 23, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auth0 Auth0 related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants