Skip to content

Add an "email change" example in the GoTrue docs#2380

Merged
kiwicopple merged 1 commit into
supabase:masterfrom
pablocubico:patch-1
Jul 16, 2021
Merged

Add an "email change" example in the GoTrue docs#2380
kiwicopple merged 1 commit into
supabase:masterfrom
pablocubico:patch-1

Conversation

@pablocubico

Copy link
Copy Markdown
Contributor

What kind of change does this PR introduce?

Docs update!

What is the current behavior?

The flow for updating a user's e-mail is not described in the current docs.

What is the new behavior?

I added an explanation of what happens after calling updateUser and clicking "Confirm" in the "Confirm Email Change" mail.

Additional context

There is no issue but there is a discussion about it.
#1763

The template for the "Confirm Email Change" e-mail and its subject is not yet available in the dashboard. Maybe create a follow-up issue to document the way of providing a template via API, and perhaps adding it to the dashboard (I know the dashboard's code is not open yet).

Keep up the good work! Supabase rocks.

@vercel

vercel Bot commented Jul 15, 2021

Copy link
Copy Markdown

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployments, click below or on the icon next to each commit.

docs – ./web

🔍 Inspect: https://vercel.com/supabase/docs/F3t2GxqRQHT7UW1MY2JsLWWwfwpZ
✅ Preview: https://docs-git-fork-pablocubico-patch-1-supabase.vercel.app

www – ./www

🔍 Inspect: https://vercel.com/supabase/www/8tpYADVZWsNhTxZmBNgsc1ZmiAdw
✅ Preview: Canceled

main – ./

🔍 Inspect: https://vercel.com/supabase/main/43gkbyfcMLmiCPpNMRqa3QdCxVWH
✅ Preview: https://main-git-fork-pablocubico-patch-1-supabase.vercel.app

@kiwicopple

Copy link
Copy Markdown
Member

Great work here @pablocubico - if you send your tshirt size and postal address to swag@supabase.io we'll send you a package

We should probably update gotrue-js to automatically detect this param in the URL.

@kiwicopple kiwicopple merged commit 6555927 into supabase:master Jul 16, 2021
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