CSC/ECE 517 Spring 2021 - E2110. Regulate changing of rubrics while projects are in progress: Difference between revisions

From Expertiza_Wiki
Jump to navigation Jump to search
No edit summary
Line 8: Line 8:
*Sai Krishna Wupadrashta Dhinakara Subrahmanya
*Sai Krishna Wupadrashta Dhinakara Subrahmanya


== Planned Changes (Design Document) ==
== Planned Work (Design Document) ==
We will write comprehensive tests and fix any bugs in the code written already for changing rubrics while projects are in progress.
 
Some known issues already have been identified:
* If you are in the active period, you get a message that if you remove a question, there should be a confirmation prompt/alert prompted to the instructor.(This is because the remove happens from a completely diff. place in the code where it is not so convenient.)
 
* Email functionality is not checked, beyond the right arguments being passed. The team had problems when calling the mailer function.
 
* ''However, reviewers can still edit their reviews, but all text for the questions has been deleted.''  '''(No, actually, these are different qq. than the reviewer answered before.)'''
 
* If you only edit qq. on the review, you can’t edit the review (if it has been submitted), but you can view it.
 
* Method find_review_period may well be redundant, though they say it is not.  Find deadlines does exist, but they say not finding a particular deadline.
 
* The method that finds the last assgt. that uses a rubric needs a method comment.
 
* As the reviews are getting deleted, this feature has to be tested thoroughly. The tests written are not thorugh and does not verify the rows deleted
 
* If Minor change, when the student hasn't submitted, what would happen?
 
* Are additional comments deleted? Might be a good idea to keep the additional comments and delete the rest of the answers. In the future, deleted based on each anser rather than the questionnaire can be an option

Revision as of 22:51, 26 March 2021

Members

  • Nicholas Himes
  • Surya Makthal
  • Jordan Boerger
  • Sai Krishna Wupadrashta Dhinakara Subrahmanya

Planned Work (Design Document)

We will write comprehensive tests and fix any bugs in the code written already for changing rubrics while projects are in progress.

Some known issues already have been identified:

  • If you are in the active period, you get a message that if you remove a question, there should be a confirmation prompt/alert prompted to the instructor.(This is because the remove happens from a completely diff. place in the code where it is not so convenient.)
  • Email functionality is not checked, beyond the right arguments being passed. The team had problems when calling the mailer function.
  • However, reviewers can still edit their reviews, but all text for the questions has been deleted. (No, actually, these are different qq. than the reviewer answered before.)
  • If you only edit qq. on the review, you can’t edit the review (if it has been submitted), but you can view it.
  • Method find_review_period may well be redundant, though they say it is not. Find deadlines does exist, but they say not finding a particular deadline.
  • The method that finds the last assgt. that uses a rubric needs a method comment.
  • As the reviews are getting deleted, this feature has to be tested thoroughly. The tests written are not thorugh and does not verify the rows deleted
  • If Minor change, when the student hasn't submitted, what would happen?
  • Are additional comments deleted? Might be a good idea to keep the additional comments and delete the rest of the answers. In the future, deleted based on each anser rather than the questionnaire can be an option