E1904 Issues related to topic deadlines: Difference between revisions

From Expertiza_Wiki
Jump to navigation Jump to search
No edit summary
No edit summary
 
(8 intermediate revisions by the same user not shown)
Line 1: Line 1:
'''The following modifications were made so far:'''
'''Submission links:'''


1. Copied in some code from the previous semester's implementation of this project (they didn't do it properly and weren't permitted to check-in)
GitHub Pull Request: https://github.com/expertiza/expertiza/pull/1401
 
YouTube Video Showing Test: https://youtu.be/b3giXbKLirk
 
 
----
 
 
'''The following modifications were made:'''
 
1. Copied in some code from the previous semester's implementation of this project (https://github.com/expertiza/expertiza/pull/1270/commits/72d97ffb5dd2279202efb2b1d329d64618403725 - they didn't do it properly and weren't permitted to check-in)


2. Successfully recreated Expertiza issue #98
2. Successfully recreated Expertiza issue #98


3. Added field auto_dropped_from_waitlist to SignedUpTeam
3. When checking whether a team needs to be removed from the waitlist, looped through all waitlisted teams on all topics for which the drop topic deadline has passed
 
4. Renamed the update_is_waitlisted method to clear_waitlisted_teams_for_topic to better reflect what the method does
 
5. Moved the code to clear the waitlists if needed to within sign_up_topic.rb so it can be called from multiple locations


4. When checking whether a team needs to be removed from the waitlist, looped through all waitlisted teams on all topics for which the drop topic deadline has passed
6. Added call to clear the waitlists if needed to assignments_controller - edit and update, so the instructor can also see the updated state with the cleared waitlists


5. Renamed the update_is_waitlisted method to clear_waitlisted_teams_for_topic to better reflect what the method does
7. Verified Expertiza issue #98 has been fixed


6. Verified Expertiza issue #98 has been fixed
8. Tested to ensure things work if it’s a staggered deadline assignment


7. Tested to ensure things work if it’s a staggered deadline assignment
9. Updated documentation on wiki


8. Updated documentation on wiki


----




'''Still yet to complete:'''
'''Unable to complete:'''


1. Refactor code implementation for fixing of Expertiza issue #98
1. Remove commit fb75d88e5519e56003eaaa56e01173c0479d2682 from our branch


2. Fix Expertiza issue #154
2. Adding field auto_dropped_from_waitlist to SignedUpTeam


3. Rather than deleting a waitlisted team from the topic altogether, unmark "is_waitlisted" and instead mark "auto_dropped_from_waitlist" to preserve the history


4. Adding the "auto_dropped_from_waitlist" to a view so the instructor could see it
5. Fix Expertiza issue #154
----




Line 46: Line 67:
- student6396 (can impersonate using instructor6 account)
- student6396 (can impersonate using instructor6 account)


----




Line 57: Line 80:
- There seems to be about a 20 minute drop topic deadline lag, some of the time
- There seems to be about a 20 minute drop topic deadline lag, some of the time


----




Line 79: Line 104:


- Click "Assignments" and then the blue plus sign (new) button
- Click "Assignments" and then the blue plus sign (new) button
- On "General" page, fill out / check ONLY the following:
- On "General" page, fill out / check ONLY the following:
     - Assignment name
     Assignment name
     - Course (select the recently created course)
     Course (select the recently created course)
     - Available to students?
     Available to students?
- On "Rubrics" page, fill out the following:
- On "Rubrics" page, fill out the following:
     - Review: "Questionnaire"
     Review: "Questionnaire"
     - Author Feedback: "Questionnaire"
     Author Feedback: "Questionnaire"
- On "Review strategy", leave all as is
- On "Review strategy", leave all as is
- On "Due dates" page, fill out the following:
- On "Due dates" page, fill out the following:
     - Round 1: Submission "Date & time"
     Round 1: Submission "Date & time"
     - Round 1: Review "Date & time"
     Round 1: Review "Date & time"
- Click "Create"
- Click "Create"


Line 95: Line 121:


- In the "Topics" tab:
- In the "Topics" tab:
     - Click "New topic"
     Click "New topic"
     - Fill out all fields (put a small number for "Number of slots")
     Fill out all fields (put a small number for "Number of slots")
     - Click Create
     Click Create


- In the "Other stuff" tab:
- In the "Other stuff" tab:
     - Click "Add Participant"
     Click "Add Participant"
     - Enter a username and click "Add" as "participant"
     Enter a username and click "Add" as "participant" (do this for multiple students - student6390, student6391, etc.)
        - (do this for mulitple students - student6390, student6391, etc.)


- Navigate to the "Editing Assignment:" page for the assignment you just created
- Navigate to the "Editing Assignment:" page for the assignment you just created
- In the "Due dates" tab:
- In the "Due dates" tab:
     - Check "Use signup deadline"
     Check "Use signup deadline"
     - Check "Use drop topic deadline"
     Check "Use drop topic deadline"
     - Fill out Drop topic "Date & time"
     Fill out Drop topic "Date & time"
- Click Save
- Click Save


- On the toolbar, hover over "Manage..."
- On the toolbar, hover over "Manage..."
- Click "Impersonate User"
- Click "Impersonate User"
- Enter a student's username who is a participant of this assignment
- Enter a student's username who is a participant of this assignment
- Click on the assignment you just created
- Click on the assignment you just created
- Click "Signup sheet"
- Click "Signup sheet"
- Click the green checkmark under "Actions"
- Click the green checkmark under "Actions"
- (Repeat the above steps until there are enough students to have one on the waitlist)
- (Repeat the above steps until there are enough students to have one on the waitlist)


- Revert the user impersonation so you're again controlling the instructor
- Revert the user impersonation so you're again controlling the instructor
- Navigate to the "Editing Assignment:" page again
- Navigate to the "Editing Assignment:" page again
- In the "Due dates" tab
- In the "Due dates" tab
     - Uncheck "Use drop topic deadline"
     Uncheck "Use drop topic deadline"
     - Check "Use signup deadline"
     Check "Use signup deadline"
     - Check "Use drop topic deadline"
     Check "Use drop topic deadline"
     - Modify the drop topic deadline so it is soon, but not immediately
     Modify the drop topic deadline so it is soon, but not immediately
- Click save
- Click save


- On the toolbar, hover over "Manage..."
- On the toolbar, hover over "Manage..."
- Click "Impersonate User"
- Click "Impersonate User"
- Enter a student's username who has signed up for a topic of this assignment
- Enter a student's username who has signed up for a topic of this assignment
- Click on the assignment you just created
- Click on the assignment you just created
- Click "Signup sheet"
- Click "Signup sheet"
- Notice how the student has the option to drop this topic
- Notice how the student has the option to drop this topic
- Also notice there is a student on the waitlist
- Also notice there is a student on the waitlist
- Allow the drop topic deadline you set to pass
- Allow the drop topic deadline you set to pass
- Refresh the page
- Refresh the page
- Notice now the student does not have the option to drop this topic
- Notice now the student does not have the option to drop this topic
     - This is working as desired
     This is working as desired
 
- Also notice there is still a student on the waitlist
- Also notice there is still a student on the waitlist
     - This is Expertiza issue #98
     This is Expertiza issue #98
     - The waitlist should be cleared when a drop topic deadline has passed
     The waitlist should be cleared when a drop topic deadline has passed

Latest revision as of 04:37, 2 April 2019

Submission links:

GitHub Pull Request: https://github.com/expertiza/expertiza/pull/1401

YouTube Video Showing Test: https://youtu.be/b3giXbKLirk




The following modifications were made:

1. Copied in some code from the previous semester's implementation of this project (https://github.com/expertiza/expertiza/pull/1270/commits/72d97ffb5dd2279202efb2b1d329d64618403725 - they didn't do it properly and weren't permitted to check-in)

2. Successfully recreated Expertiza issue #98

3. When checking whether a team needs to be removed from the waitlist, looped through all waitlisted teams on all topics for which the drop topic deadline has passed

4. Renamed the update_is_waitlisted method to clear_waitlisted_teams_for_topic to better reflect what the method does

5. Moved the code to clear the waitlists if needed to within sign_up_topic.rb so it can be called from multiple locations

6. Added call to clear the waitlists if needed to assignments_controller - edit and update, so the instructor can also see the updated state with the cleared waitlists

7. Verified Expertiza issue #98 has been fixed

8. Tested to ensure things work if it’s a staggered deadline assignment

9. Updated documentation on wiki




Unable to complete:

1. Remove commit fb75d88e5519e56003eaaa56e01173c0479d2682 from our branch

2. Adding field auto_dropped_from_waitlist to SignedUpTeam

3. Rather than deleting a waitlisted team from the topic altogether, unmark "is_waitlisted" and instead mark "auto_dropped_from_waitlist" to preserve the history

4. Adding the "auto_dropped_from_waitlist" to a view so the instructor could see it

5. Fix Expertiza issue #154




Accounts used for testing:

- instructor6/password

- student6390 (can impersonate using instructor6 account)

- student6391 (can impersonate using instructor6 account)

- student6392 (can impersonate using instructor6 account)

- student6393 (can impersonate using instructor6 account)

- student6394 (can impersonate using instructor6 account)

- student6395 (can impersonate using instructor6 account)

- student6396 (can impersonate using instructor6 account)




Other related issues found:

- When editing an assignment's due dates, the drop topic deadline won't display unless the signup deadline is displayed

- When a student is on the waitlist for more than one topic, only the row for one of the topics is grey

- There seems to be about a 20 minute drop topic deadline lag, some of the time




To recreate Expertiza issue #98, perform the following steps

- Login as username = instructor6, password = password

- Click "Manage...", then "Courses"

- Make sure "Courses" is selected when the page loads

- Click the blue plus sign (new) button

- On the "New Course" page, fill out / check the following:

   Institution Name
   Course Name
   Course Directory
   Course Information
   Private course

- Click "Create"

- Click "Assignments" and then the blue plus sign (new) button

- On "General" page, fill out / check ONLY the following:

   Assignment name
   Course (select the recently created course)
   Available to students?

- On "Rubrics" page, fill out the following:

   Review: "Questionnaire"
   Author Feedback: "Questionnaire"

- On "Review strategy", leave all as is - On "Due dates" page, fill out the following:

   Round 1: Submission "Date & time"
   Round 1: Review "Date & time"

- Click "Create"

- If not sent to "Editing Assignment:" page, navigate to it.

- In the "Topics" tab:

   Click "New topic"
   Fill out all fields (put a small number for "Number of slots")
   Click Create

- In the "Other stuff" tab:

   Click "Add Participant"
   Enter a username and click "Add" as "participant" (do this for multiple students - student6390, student6391, etc.)

- Navigate to the "Editing Assignment:" page for the assignment you just created

- In the "Due dates" tab:

   Check "Use signup deadline"
   Check "Use drop topic deadline"
   Fill out Drop topic "Date & time"

- Click Save

- On the toolbar, hover over "Manage..."

- Click "Impersonate User"

- Enter a student's username who is a participant of this assignment

- Click on the assignment you just created

- Click "Signup sheet"

- Click the green checkmark under "Actions"

- (Repeat the above steps until there are enough students to have one on the waitlist)

- Revert the user impersonation so you're again controlling the instructor

- Navigate to the "Editing Assignment:" page again

- In the "Due dates" tab

   Uncheck "Use drop topic deadline"
   Check "Use signup deadline"
   Check "Use drop topic deadline"
   Modify the drop topic deadline so it is soon, but not immediately

- Click save

- On the toolbar, hover over "Manage..."

- Click "Impersonate User"

- Enter a student's username who has signed up for a topic of this assignment

- Click on the assignment you just created

- Click "Signup sheet"

- Notice how the student has the option to drop this topic

- Also notice there is a student on the waitlist

- Allow the drop topic deadline you set to pass

- Refresh the page

- Notice now the student does not have the option to drop this topic

   This is working as desired

- Also notice there is still a student on the waitlist

   This is Expertiza issue #98
   The waitlist should be cleared when a drop topic deadline has passed