-
Notifications
You must be signed in to change notification settings - Fork 57
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
Email Verification Added #42
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- After the user signs up, instead of showing the user a dialogue box saying Proceed with login. It would be better if it informed that a verification email is sent to your registered email id. Verify and Proceed to login.
- Could you add the padding to the "Hello There ." text in the login screen so that it matches the UI in the readme.
Paddding there means some spaces from uppersie of that text to look good right just like README.....????? I will make changes within an hour....thanks @pratyushmp |
lib/sign_up.dart
Outdated
@@ -250,7 +250,7 @@ class SignUpScreenState extends State<SignUpScreen> { | |||
style: TextStyle(color: Colors.white), | |||
), | |||
content: Text( | |||
"Please Log-In To Continue", | |||
"Verification Link Send to Your Registered Mail\nPlease Verify and Log-In", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
change "Send" to "Sent".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Resolved
Already Made changes..... please check it @pratyushmp Hope you review it and merge it @pratyushmp @infiniteoverflow @nirala96 |
Good Job @SamarpanCoder2002 . |
Thank u @pratyushmp ... I requested in other issue about adding profile picture.....kindly check this |
Description :
Fixes #38 :
Replace
#38
with the issue number which is fixed in this PRHave you read the Contributing Guidelines on Pull Requests?
Type of change :
Please delete/edit options that are not relevant.
Checklist :
If this pull request ok, please review and merge it..... Thanks @pratyushmp