search

LetsTagOn / LetsTagOnThoughtClan

This is for development of a social media platform to promote volunteering and to bring together various players in the development sector. The technology stack is Java Script, Bootstrap UI framework, JSpring, Node.JS and MySQL. Looking for passionate developers to contribute to this platform development. Reach out to cskoulagi@gmail.com.
1 stars 0 forks source link

UI: High: Fixing Messages #1

Open ChetanaKoulagi opened 6 years ago

ChetanaKoulagi commented 6 years ago
  1. If an email id is registered already, it gives error "oops something went wrong". It should just say this email is already registered.
  2. Other message updates sent as an attachment over email.
Viswa-Vinod commented 6 years ago

"oops something went wrong" is the title of the modal (the error message box). The actual error message will be in the body of the modal. The title of the modal should be left generic. So I changed it to 'Registration Error'. I believe what you are asking for - 'email is already registered' - should appear in the body of the modal. Please let me know what you are seeing in the body of the message.

As for the rest of the messaging changes sent through email, points 2,3 and 4 - are backend java changes. Backend engineer should handle that. I have fixed 1, 5 and 6. I have pushed all my changes to vinod/featurebranch.

ChetanaKoulagi commented 6 years ago

Yes, you are right on the 'email is already registered' should appear in the body of the modal.

For others, will ask to be fixed at backend.

Viswa-Vinod commented 6 years ago

Does it not appear now? What appears in the body of the modal now?

Viswa-Vinod commented 6 years ago

According to the code, the modal simply displays whatever message is received from the backend in the body of the modal. So, if you want the body of the modal to have the right message, you may want to check with the backend engineer.

ChetanaKoulagi commented 6 years ago

Ok. Will check at backend.

sathyab28 commented 6 years ago

Shilpa's backend changes has been pushed