GOOGLE-83: Better Error handling experience for the users

Activity

CONTRIB-15 1

Keyboard shortcuts  
Summarize the review outcomes (optional)
 
#permalink

Details

Warning: no files are visible, they have all been filtered.
Participant Role Time Spent Comments Latest Comment
Author 0m    
Reviewer - Complete 3m 1 Duplicate of Contrib-14, so not reviewing.
Reviewer - 0% reviewed      
Total   3m 1  
#permalink

Objectives

If there are any exceptions in the code the users are not notified about this. grace full error handling should be implemented in the following cases:

1) If for any reason when linking/unlinking of the folder to the site failed the user is not shown with useful error message only logs show the error.
2) If for any reason when inserting/removing permission to the folder failed the user is not shown with useful error message only logs show the error.
3) If an instructor creates a site and nobody is in the site roster. When instructor shares his Google drive folder in this case after sharing the view is not changed to the Shared view and remain in the create and share view. But actually every thing has gone correct. after hitting a refresh button the page is opened in the shared view.
4) A warning messing is shown in the logs when a student logs in to the site when no folder is being shared. The message is rather confusing. From the UI the Student see "No folder has been shared with the site message"- which is good.
"Unauthorized attempt to acces JSP page LinkFolder requiring roles [Instructor] by null <test4@collab.its.umich.edu> with roles [Learner]"
5) During the launch of the Google-Drive-LTI tool(i.e POST) if some reason error occurs then the error are being presented to the user but the the error messages need to be coming from the properties file supporting localization.

Branches in review

#permalink

Issues Raised From Comments

Key Summary State Assignee
#permalink

General Comments

David Haines

Duplicate of Contrib-14, so not reviewing.

Duplicate of Contrib-14, so not reviewing.

/umich/.../bundle/googleDriveLTIProps.properties Changed
/umich/google/.../google/GoogleLtiServlet.java Changed
/umich/.../lti/TcSiteToGoogleStorage.java Changed
/umich/google/.../webapps/js/googleDriveLti.js Changed

Review updated: Reload | Ignore | Collapse

You cannot reload the review while writing a comment.

Create issue

X
Assign To Me

Log time against