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 toasty on signup page #299

Merged
merged 3 commits into from
Dec 6, 2024
Merged

fix toasty on signup page #299

merged 3 commits into from
Dec 6, 2024

Conversation

Mugisha146
Copy link
Contributor

@Mugisha146 Mugisha146 commented Dec 6, 2024

PR Description

Fixes the toasty issue on the signup page encountered during the Pre-demo

Description of tasks that were expected to be completed

This pull request addresses the issue where messages (toasts) weren't displayed on the signup page, specifically when using existing email addresses or phone numbers etc

How has this been tested?

  • Clone repo
  • Checkout branch fx-toasty
  • Make sure that you set the environment variables referring to .env.example
  • Run npm install, then npm run dev to check if the app is running successfully
  • Navigate to signup page then try to signup by using existing email (ex: [email protected]) or existing telephone etc
  • You should see a toast message indicating the error.

Screenshoot

Screenshot 2024-12-06 at 14 17 17

Copy link

vercel bot commented Dec 6, 2024

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

Name Status Preview Comments Updated (UTC)
atlp-devpulse-fn ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 6, 2024 1:36pm

Copy link

codeclimate bot commented Dec 6, 2024

Code Climate has analyzed commit 52d6893 and detected 0 issues on this pull request.

View more on Code Climate.

@ncutixavier ncutixavier merged commit a2948a4 into develop Dec 6, 2024
7 checks passed
@ncutixavier ncutixavier deleted the fx-toasty branch December 6, 2024 14:00
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.

5 participants