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

Bug : Simplify error messages for user at Talawa User Portal #2464

Closed
bandhan-majumder opened this issue Nov 23, 2024 · 14 comments
Closed

Bug : Simplify error messages for user at Talawa User Portal #2464

bandhan-majumder opened this issue Nov 23, 2024 · 14 comments
Assignees
Labels
bug Something isn't working feature request invalid This doesn't seem right

Comments

@bandhan-majumder
Copy link
Contributor

Describe the bug
In Talawa User portal when the user tries to update information with empty or invalid data, error in not user friendly way appears. Makes it hard to understand the user what required fields they are missing or giving an invalid data into.

To Reproduce
Steps to reproduce the behavior:

  1. Go to Talawa User Portal
  2. Go to settings
  3. Submit data in invalid or empty format
  4. Click on save changes

Expected behavior
Error should come one by one in a user friendly format.

Actual behavior

Variable "$phoneNumber" got invalid value ""; Value is not a valid phone number of the form +17895551234 (7-15 digits): Variable "$grade" got invalid value ""; Value "" does not exist in "EducationGrade" enum. Variable "$empStatus" got invalid value ""; Value "" does not exist in "EmploymentStatus" enum. Variable "$maritalStatus" got invalid value ""; Value "" does not exist in "MaritalStatus" enum.

Screenshots
image

Additional details
Add any other context or screenshots about the feature request here.

Potential internship candidates

Please read this if you are planning to apply for a Palisadoes Foundation internship

@bandhan-majumder bandhan-majumder added the bug Something isn't working label Nov 23, 2024
Copy link

Congratulations on making your first Issue! 🎊 If you haven't already, check out our Contributing Guidelines and Issue Reporting Guidelines to ensure that you are following our guidelines for contributing and making issues.

@bandhan-majumder
Copy link
Contributor Author

bandhan-majumder commented Nov 23, 2024

I would like to work on this issue. And please clarify my doubt that my changes should be pushed against develop or develop-postgres branch cause talawa-api's develop-postgres branch is not ready yet.

@palisadoes
Copy link
Contributor

  1. Will your solution work in all cases in the app, or just this particular one?
  2. Will you have translations for your error(s)?

@bandhan-majumder
Copy link
Contributor Author

bandhan-majumder commented Nov 23, 2024

  1. As the submit handler of this is different, it will work for this section only. But if we need, we can easily make it work for other handlers as well.
  2. Yes

@palisadoes
Copy link
Contributor

palisadoes commented Nov 24, 2024

Is a universal handler feasible?

@bandhan-majumder
Copy link
Contributor Author

Wherever the error occurs, if we do tErrors('specificErrorName') it will show the user friendly version of the specific error message. Here is an example of how existing talawaApiUnavailable error message is present in different places but maps to the same error message.

presents in places:
image
maps to the same:
image

@palisadoes
Copy link
Contributor

Will you be able to do the code updates for all the cases with tests to resolve this issue?

@bandhan-majumder
Copy link
Contributor Author

bandhan-majumder commented Nov 24, 2024

I have solved this particular issue locally. Should I raise a PR and then you can suggest changes/modifications it needs!!

@gurramkarthiknetha
Copy link
Contributor

gurramkarthiknetha commented Nov 24, 2024

i fixed this issue can i raise a PR

@gurramkarthiknetha
Copy link
Contributor

image i sloved this issue

@gurramkarthiknetha
Copy link
Contributor

gurramkarthiknetha commented Nov 24, 2024

image

could you please assign this issue to me i will raise a pr

Copy link

github-actions bot commented Dec 9, 2024

This issue did not get any activity in the past 10 days and will be closed in 180 days if no update occurs. Please check if the develop branch has fixed it and report again or close the issue.

@github-actions github-actions bot added no-issue-activity No issue activity and removed no-issue-activity No issue activity labels Dec 9, 2024
@bandhan-majumder
Copy link
Contributor Author

bandhan-majumder commented Dec 11, 2024

I am unassigning myself from this as nobody is responding. There is a PR open at #2500 . I am unable to get new issues. Happy to look at it if anything needs change.

@varshith257
Copy link
Member

Fixed by #2500

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working feature request invalid This doesn't seem right
Projects
None yet
Development

No branches or pull requests

4 participants