CSC/ECE 517 Spring 2023 - E2302: Difference between revisions

From Expertiza_Wiki
Jump to navigation Jump to search
No edit summary
Line 18: Line 18:


== Our Changes ==
== Our Changes ==
Upon review of previous PRs, we found that the work was thorough, albeit a few test cases were failing, and comments and methods could be refactored better. Based on the prompts we got, we focussed on refactoring the create method of assignments controller, and assignments form, along with the refactoring of sidekiq-related files.
 
Below is the summary of all the changes as part of this PR.

Revision as of 00:39, 23 March 2023

E2302. Refactoring the Delayed Mailer and Scheduler

Topic Overview

Background


Prior Work

History: Previous projects

* E2253 [1]
* E2144 [2]
* E2273 [3]

Our Changes

Upon review of previous PRs, we found that the work was thorough, albeit a few test cases were failing, and comments and methods could be refactored better. Based on the prompts we got, we focussed on refactoring the create method of assignments controller, and assignments form, along with the refactoring of sidekiq-related files.

Below is the summary of all the changes as part of this PR.