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

login error message #161

Closed
wants to merge 3 commits into from
Closed

Conversation

Omsurvase24
Copy link

Login Page Error message issue solved
Issue number #83
When entered wrong email or password or if there is any other error it showed error : Request failed with status code 401
Which for common user was not friendly.
Solved this and made it so that the common user knows what the problem exactly is, making it more user friendly.

Please review.

Copy link

vercel bot commented May 29, 2024

@Omsurvase24 is attempting to deploy a commit to the Alogovengers' projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Awesome work, @Omsurvase24! 🎉 Thank you for your contribution and efforts to enhance Hairify.

We will review your changes soon and provide feedback.

Please ensure that you have followed the contributing guidelines for a smooth review process.

@@ -1,7 +1,7 @@
"use client";
import React, { useState } from "react";
import Link from "next/link";
import axios from "axios";
import axios, { AxiosError } from "axios";
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove this unnecessary import.

@Omsurvase24
Copy link
Author

Removed the imports that were not required and even squashed the commits. I don't know why is it not showing though.

@afeefuddin
Copy link
Collaborator

Squash your commits and also remove the merge commit.

@afeefuddin afeefuddin closed this Dec 10, 2024
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