CSC/ECE 517 Spring 2021 - E2106. Fix view in student task/list page

From Expertiza_Wiki
Jump to navigation Jump to search

This page details project documentation for the CSC/ECE 517 Spring 2021, "E2106 Fix view in student task/list page" project.


Background

The student_task/list page is the page displayed to a student after logging in into expertiza. It has mainly two div(s), one to show the upcoming tasks along with some important information, another div is to display all the assignments and their information in which the student is participating. We have to modify the view to remove extraneous information and make it clearer and more concise.

Mentor

Saurabh Shingte, svshingt@ncsu.edu

Team Members

  • Sharath Kumar (svkumar2@ncsu.edu)
  • Nikhil Sundaraswamy (nsundar@ncsu.edu)

Issues

  1. Modify the view to remove extraneous information and make it clearer and more concise
  2. Add checkbox to Publishing Rights column, when checked the permission_granted boolean variable is set to true
  3. In general, no column needs to be shown if there are no values in it. This would apply to “Badges” if no assignments in the list contain badges.
  4. Also, it doesn’t make sense to have a column for Review Grade, and not for the grade for submitted work (“Submission Grade”).
  5. We should consider the cases where a student might be enrolled in assignments from more than one course. In this case, assignments should be grouped by course, with the course name given before the listing of the first assignment. (This allows for the column for course to be removed, making the display more compact.

Functionality Testing

Log in to Expertiza as any of the students mentioned below. Without the changes the original view is as shown below.

  • instructor6
  • student7487
  • student7490
  • student7496

Password for logging in for any student/instructor is : password

Once logged into the page click on "Assignments" tab. A modified view is observed as compared to the earlier view

To address issue 1 we added CSS style properties and removed the unnecessary space between two divs by using HTML comment.

The modified view is as shown below

To address issue 2, we did the following modifications

  • In app/assests/javascripts/submissions.js, we have added an ajax call to update the permission_granted boolean variable in Participant table

  • In app/controller/student_task_controller.rb, we are defining a method named permission_granted which routes the action from checkbox event

  • app/views/student_task/list.html.erb, we have added code to include an information banner on Publishing Rights column and checkboxes

  • In config/routes.rb, we have added a functionality to route the checkbox event to permission_granted action in student_task_controller.rb

After all proposed modifications, we see the below shown view

Test Plan

Since this project involved mostly UI changes it was not necessary to write test cases for the same. So there is no test plan


Links