Skip to content
This repository has been archived by the owner on May 20, 2023. It is now read-only.

Improve existing planner_move to handle co-requisites #164

Closed
Letm3through opened this issue Apr 3, 2019 · 0 comments · Fixed by #197
Closed

Improve existing planner_move to handle co-requisites #164

Letm3through opened this issue Apr 3, 2019 · 0 comments · Fixed by #197
Assignees
@Letm3through
Labels
type.Enhancement An enhancement to an existing story
Milestone
v1.4

Comments

@Letm3through
Copy link

Letm3through commented Apr 3, 2019 •
edited
Loading

Currently, planner_move command does not handle co-requisites. Hence, when the module in the degree plan with co-requiste is moved, only that module will be moved and other modules with the module's co-requisite will not be moved.

Hence, let's update planner_move logic to handle the co-requisites so that when the module is moved, other modules with the module's co-requisite will be moved too.
@Letm3through Letm3through changed the title PlannerMove Code case sensitive PlannerMove Code Corequisite Apr 3, 2019
@Letm3through Letm3through changed the title PlannerMove Code Corequisite Improve existing planner_move to handle co-requisites Apr 3, 2019
@Letm3through Letm3through added the type.Enhancement An enhancement to an existing story label Apr 3, 2019
@Letm3through Letm3through added this to the v1.4 milestone Apr 3, 2019
@Letm3through Letm3through self-assigned this Apr 3, 2019
@Letm3through Letm3through mentioned this issue Apr 14, 2019
Merged
Creastery added a commit that referenced this issue Apr 14, 2019
@Creastery
[#164] Logic/commands: update planner_move to handle co-requisites (#…
80dd0d6 
…197)

The application allows the user to set co-requisites to modules. 
However, `planner_move` does not handle co-requisites. Hence, the user 
has to move the codes that shares co-requisites manually.

Let's enhance `planner_move` command by allowing it to handle 
co-requisites so that when a module is moved, its co-requisites are 
moved along if they exist in the degree plan.

In addition, since there has been update on `ModuleList` to store the 
semesters the module is offered in, let's also enhance `planner_move` 
command to check if the module is offered in the semester the user wants
to move to.

The followings are the changes in the process of the enhancement:

  [1/3] Logic: update 'planner_move' to handle co-requisites
  [2/3] Model: add method to help in carrying out handling of co-requisites
  [3/3] Test: update test cases for 'planner_move' command
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Assignees

@Letm3through Letm3through

Labels
type.Enhancement An enhancement to an existing story
Projects
None yet
Milestone
v1.4
Development

Successfully merging a pull request may close this issue.

Logic: update planner_move to handle co-requisites Letm3through/main
1 participant
@Letm3through