Skip to content

Noel - #7

Open
noellimx wants to merge 264 commits into
nus-tic2002-2021:masterfrom
nlx-academic:master
Open

Noel#7
noellimx wants to merge 264 commits into
nus-tic2002-2021:masterfrom
nlx-academic:master

Conversation

@noellimx

@noellimx noellimx commented Aug 31, 2021

Copy link
Copy Markdown

No description provided.

- add tests
- add ToDo extends Task
- add TaskManager with:
  - TaskList as object attribute of TaskManager to store collection of tasks
  - create new ToDos object
- add Command (and its response) capability for:
 - adding task
 - list tasks of taskManagers
- UxManager
 - add support for new reponse type 1) add todo and 2) list tasks
- Custom Command Factory to interprete textCommand
- tick off todo
- update changelog
- edit doc
- set objectives
- setup test
- add command capabilities for 1) mark tasks stored in TaskManager objects as done 2) task not found response
- add support for:
 - task update done status response
 - validating textCommand requests for 1) mark tasks as done 2) add task todo
 - OrchestratorLevel03 with customised CommandFactory
- tick off todo, update changelog
- set objectives
- setup test skeleton
- mark add a ToDo as done
- setup test for adding deadline
- add deadline extends Task
- add command support for creating deadlines
- add response for adding deadline
- add support for validating add deadline request
- setup test for adding event
- add Event extends Task
- add command support for creating event
- add response for adding event
- add support for validating add event request
- setup test for default command - CommandUnknownRequest
- unrecognised command will execute CommandUnknownRequest
- added clarity: terminate means process ends, exit loop means scanner will stop scanning for next command.
- tick off todos
- update changelog and readme
@noellimx
noellimx force-pushed the master branch 3 times, most recently from efe30e6 to 0a27958 Compare September 21, 2021 01:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant