Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix/password recovery form not shown #583

Merged
merged 3 commits into from
May 11, 2023

Conversation

raphael-arce
Copy link
Contributor

No description provided.

Signed-off-by: Raphael Arce <raphael.arce@ts.berlin>
Signed-off-by: Raphael Arce <raphael.arce@ts.berlin>
Signed-off-by: Raphael Arce <raphael.arce@ts.berlin>
@vercel
Copy link

vercel bot commented May 11, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
giessdenkiez-de ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 11, 2023 3:09pm

@socket-security
Copy link

New dependency changes detected. Learn more about Socket for GitHub ↗︎


👍 No new dependency issues detected in pull request

Bot Commands

To ignore an alert, reply with a comment starting with @SocketSecurity ignore followed by a space separated list of package-name@version specifiers. e.g. @SocketSecurity ignore foo@1.0.0 bar@* or ignore all packages with @SocketSecurity ignore-all

Pull request alert summary
Issue Status
Install scripts ✅ 0 issues
Native code ✅ 0 issues
Bin script shell injection ✅ 0 issues
Unresolved require ✅ 0 issues
Invalid package.json ✅ 0 issues
HTTP dependency ✅ 0 issues
Git dependency ✅ 0 issues
Potential typo squat ✅ 0 issues
Known Malware ✅ 0 issues
Telemetry ✅ 0 issues
Protestware/Troll package ✅ 0 issues

📊 Modified Dependency Overview:

⬆️ Updated Package Version Diff Added Capability Access +/- Transitive Count Publisher
@supabase/auth-ui-react@0.4.2 0.3.5...0.4.2 None +7/-6 kiwicopple
@supabase/auth-helpers-nextjs@0.6.1 0.5.6...0.6.1 None +8/-6 phamhieu1998

@raphael-arce
Copy link
Contributor Author

@ff6347 could you please test this locally? my last tests were all working, but I'm not trusting it

@ff6347
Copy link
Collaborator

ff6347 commented May 11, 2023

@raphael-arce Yes I will. Does it work in the deploy preview?

@raphael-arce
Copy link
Contributor Author

@ff6347 No, I'm getting redirected to the staging url, and not the current branch preview url

@ff6347
Copy link
Collaborator

ff6347 commented May 11, 2023

Nope. I get directly to /profile and don't get the recovery form :/

@ff6347
Copy link
Collaborator

ff6347 commented May 11, 2023

ah okay. I need to change the env var?

@ff6347
Copy link
Collaborator

ff6347 commented May 11, 2023

Now I land on the password-reset but I don't get the form. It is stuck in loading state.

image

@ff6347
Copy link
Collaborator

ff6347 commented May 11, 2023

The recovery does not fire when I come in over that url

Copy link
Collaborator

@ff6347 ff6347 left a comment

Choose a reason for hiding this comment

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

LGTM. We need to test this in the staging env

@ff6347 ff6347 merged commit fb9ffe8 into feat/auth-via-supabase May 11, 2023
@ff6347 ff6347 deleted the fix/password-recovery-form-not-shown branch May 11, 2023 16:10
ff6347 added a commit to technologiestiftung/giessdenkiez-de-postgres-api that referenced this pull request May 12, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants