-
Notifications
You must be signed in to change notification settings - Fork 5
[$20] Get js error and crash app after searching and click on "Create" button #116
Comments
valid bug. Related to #93, but in this case, it appears that the group name should be more than 3 characters |
@callmekatootie ideally error handling on the groups API would help here, but we can also validate on the front end if only the 3 character minimum are the issue. |
Ok. I will go with the former, which will help debug issues better (and even for the front end user to understand their errors) Expected: For both Add Group modal and Groups Tab page, when we make a request to create a new group, we get back an error message from the groups api (if there is an error) - display the error message using the browser's |
Update - we do display error now. But only specific errors. The expected fix is that if the api request returns with an error status code AND has a message attribute, display the message attribute value in the alert() function. Example to test with - try entering a group name more than 150 characters or less than 3 characters... |
Contest https://www.topcoder.com/challenges/30134137 has been created for this ticket. |
Contest https://www.topcoder.com/challenges/30134137 has been updated - it has been assigned to genycopedison. |
Payment task has been updated: https://software.topcoder.com/review/actions/ViewProjectDetails?pid=30134137 |
Reproduction Steps:
Screenshot/screencast:
https://drive.google.com/drive/folders/1taqM4PrT2NNP4740QY1d0Rb9ixCJi6aN?fbclid=IwAR2pD3gYGvZcsncY7T4ewgF1IEi-61WkcDjy_Rs7N6WIH86KnWvGCSpOFdU
Actual Result: Get js error and crash app after searching and click on "Create" button
Expected Result: Should work well without any broken.
Device/OS: Laptop | Windows 10 (64bit)
Browser Information: IE11 , Google Chrome 83.0.4103.116, Firefox 78.0.2
The text was updated successfully, but these errors were encountered: