Code Snippet Mistake on the Supabase Docs

[spaansba]

Hi all,

I came across the following snippet in the Supabase docs on the "Send a password reset request" section:

/**
 * Step 1: Send the user an email to get a password reset token.
 * This email contains a link which sends the user back to your application.
 */
const { data, error } = await supabase.auth
  .resetPasswordForEmail('[email protected]')

/**
 * Step 2: Once the user is redirected back to your application,
 * ask the user to reset their password.
 */
 useEffect(() => {
   supabase.auth.onAuthStateChange(async (event, session) => {
     if (event == "PASSWORD_RECOVERY") {
       const newPassword = prompt("What would you like your new password to be?");
       const { data, error } = await supabase.auth
         .updateUser({ password: newPassword })

       if (data) alert("Password updated successfully!")
       if (error) alert("There was an error updating your password.")
     }
   })
 }, [])

In the code snippet
if (data) alert("Password updated successfully!")
will always be hit even though the password did not get updated since data looks as follows:

const data: {
    user: User;
} | {
    user: null;
}