You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
When a user submits an empty comment, the application currently shows a technical error message: "Comment validation failed: text: Path 'text' is required." This message is intended for developers and is not suitable for users. It should be replaced with a more user-friendly message like "Please enter a comment before submitting." The original technical message should be kept in the logs for developer reference, but not displayed to end-users.
To Reproduce
Steps to reproduce the behavior:
Go to the comment section.
Leave the comment field empty.
Submit the comment.
Observe the technical error message displayed.
Expected behavior
A user-friendly error message like "Please enter a comment before submitting."
Actual behavior
A technical error message: "Comment validation failed: text: Path 'text' is required."
Screenshots
Before Fix:
After Fix:
Additional details
I've implemented a fix locally by updating the error handling logic to show a more appropriate message to the users while keeping the developer-specific message for internal logs.
Potential internship candidates
Please read this if you are planning to apply for a Palisadoes Foundation internship
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.
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.
Describe the bug
When a user submits an empty comment, the application currently shows a technical error message: "Comment validation failed: text: Path 'text' is required." This message is intended for developers and is not suitable for users. It should be replaced with a more user-friendly message like "Please enter a comment before submitting." The original technical message should be kept in the logs for developer reference, but not displayed to end-users.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
A user-friendly error message like "Please enter a comment before submitting."
Actual behavior
A technical error message: "Comment validation failed: text: Path 'text' is required."
Screenshots
![Screenshot 2025-01-05 015249](https://private-user-images.githubusercontent.com/141390434/400164497-9738c469-31e8-444b-a629-f6a435f56662.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzg4Mzg3MjgsIm5iZiI6MTczODgzODQyOCwicGF0aCI6Ii8xNDEzOTA0MzQvNDAwMTY0NDk3LTk3MzhjNDY5LTMxZTgtNDQ0Yi1hNjI5LWY2YTQzNWY1NjY2Mi5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUwMjA2JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MDIwNlQxMDQwMjhaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT0wOTQyNzQyNDEzOWE5YWVmNWJhZjNkNTlkYWNjZDQyZWRlMTA3MjcxZDY1MzFjODdmYTljOWU4NGI1OGRhZmM3JlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.ceO_fg__Yjbs58M0A9f2J6vvqycxSFO79eFVOCTjUYs)
![Screenshot 2025-01-05 021900](https://private-user-images.githubusercontent.com/141390434/400164505-799999ae-792a-46ed-8b48-bdb69bade908.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzg4Mzg3MjgsIm5iZiI6MTczODgzODQyOCwicGF0aCI6Ii8xNDEzOTA0MzQvNDAwMTY0NTA1LTc5OTk5OWFlLTc5MmEtNDZlZC04YjQ4LWJkYjY5YmFkZTkwOC5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUwMjA2JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MDIwNlQxMDQwMjhaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1lZjg3ZTAxNmU5OTk1MzU1MmVmODc0MGJlZGFkMzQ3MDdmNTdiZTM0YjY1NzZmODg1NDRiZDA5MjE4YmI1YWY0JlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.seHbhcfAkuOXQNDag6b12vURfiiPjKJwKJRRu_13ipc)
Before Fix:
After Fix:
Additional details
I've implemented a fix locally by updating the error handling logic to show a more appropriate message to the users while keeping the developer-specific message for internal logs.
Potential internship candidates
Please read this if you are planning to apply for a Palisadoes Foundation internship
The text was updated successfully, but these errors were encountered: