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

fix: add createUser param to GenerateLink #1116

Closed
wants to merge 2 commits into from

Conversation

J0
Copy link
Contributor

@J0 J0 commented May 24, 2023

What kind of change does this PR introduce?

What is the current behavior?

See: supabase/auth-js#358

What is the new behavior?

Introduces an additional parameter to GenerateLink which allows user to generate a magic link. Further thought is needed, leaving as draft for now.

@J0 J0 closed this Aug 3, 2023
@activenode
Copy link

I'm slightly confused about this. Why was this closed?

@J0
Copy link
Contributor Author

J0 commented Mar 5, 2024

Hm yeah let me re-open this one. Thanks for flagging

@J0 J0 reopened this Mar 5, 2024
@J0
Copy link
Contributor Author

J0 commented Mar 5, 2024

It's been a while so I can't recall but I think the initial hesitation was because generateLink is an admin method and probably needs less guard rails like shouldCreateUser

I'll think about it + check in with team and get back. If you'd like to share more about potential use cases please feel free to do so

@J0
Copy link
Contributor Author

J0 commented Mar 5, 2024

Ah I think to preserve existing behaviour shouldCreateUser needs to default to true which is a little cumbersome at the moment.

I'll likely revisit after attending to core priorities but for now will have to trouble you to make use of a trigger or other checks.

Let me know if there are any suggestions though

@J0 J0 closed this Mar 5, 2024
@J0 J0 deleted the j0/add_should_create_user_option branch November 21, 2024 08:08
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