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

Toast functionality and contact section UI done #456

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

avleen123
Copy link

Related Issue

Older version : Contact Section UI was not upto the mark and there was no toast functionality.

Closes #395

Description

New Version: I have modified the contact section and applied the colors which match with website UI. Further more I implemented toast functionality which will show error messages and success message accordingly. You can see the attached screenshots for reference. I have created a PR please have a look!

I am raising another issue for smtp js to send email to designated email of eduhub please assign that task to me. I will like to carry out it further and make it full fledged working contact section

Screenshots

Modified_contact_form
empty_field_toast
wrong_email_toast
Successful_toast

Checklist

  • My code adheres to the established style guidelines of the project.
  • I have included comments in areas that may be difficult to understand.
  • My changes have not introduced any new warnings.
  • I have conducted a self-review of my code.

@avleen123
Copy link
Author

avleen123 commented Jul 13, 2023

Please check it now. I did the needful, now it is okay to merge

@avleen123
Copy link
Author

avleen123 commented Jul 14, 2023

Can you please merge this PR. I am not getting option at my end to merge.

@avleen123
Copy link
Author

avleen123 commented Jul 15, 2023

@SahityaRoy & @Viveksati5143 can you please merge the PR Toast functionality and contact section UI done #456. It has been approved by the reviewer, i am not getting the merge option at my end. that's why can you do merge please.

@avleen123 avleen123 closed this Aug 7, 2023
@avleen123 avleen123 reopened this Aug 7, 2023
@avleen123
Copy link
Author

avleen123 commented Aug 11, 2023

I have resolved the merge conflict please review and merge it under GSSOC'23. Have a look @Viveksati5143 @Athi223 @bLaCkwEw @ShahzaibJutt

Copy link
Contributor

@Viveksati5143 Viveksati5143 left a comment

Choose a reason for hiding this comment

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

Looks good!!

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.

Contact Section + Toast Message
2 participants