Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update user guide to include task find #101

Merged
merged 6 commits into from
Oct 28, 2021

Conversation

mingyi456
Copy link

No description provided.

* 'master' of https://github.com/AY2122S1-CS2103-F09-2/tp:
  Fix typo
  Update DG to include `FindTaskCommand` and add use cases
  Refactor Task::getIsDone to Task::isDone
  Fix checkstyle
  Fix bugs reappearing due to force push
  Refactor EditTaskDialog
  Fix bugs
  Implement add task GUI
* 'master' of https://github.com/AY2122S1-CS2103-F09-2/tp:
  Refactor GUI
  Add JavaDocs for ModelManager methods
  Update UG
  Fix tests
  Fix checkstyle
  Fix task immutability, relevant bug
  Add GUI
  Setup storage, GUI preliminaries
  Update Contacts when GUI is changed
  Setup contact add, edit
* 'master' of https://github.com/AY2122S1-CS2103-F09-2/tp:
  Fix checkstao for realsies
  Fix checkstyle
  Fix merge
  Add optional stuff
  Change Optional in FindTaskCommand
  Add undone message
  Fix compilation error
  Add undo for FindTaskCommand
  Add short User Guide for undo
  Fix redundant code
  Fix checkstyle
  Fix purge undo
  Add undo for purge, cleanup undo
  Trashed branch
* 'master' of https://github.com/AY2122S1-CS2103-F09-2/tp: (25 commits)
  Change test cases to use Timestamp.tryparse
  Fix coding standard violation
  Remove unused import
  Add ArgumentContainsSlashException
  merge upstream
  Fix GUi checkbox bug
  Fix checkstyle
  Fix even more merge errors
  Update timestamp overdue GUI
  Remove unused variable
  Update UserGuide on command parsing logic
  Rework command parsing logic
  Use LocalDate for Timestamp
  Fix test cases
  Fix checkstyle
  Update UI for timestamp
  Update storage loading for timestamp
  Add regex validation for timestamp
  Update parsers to accommodate Parsing Exception
  Merge current master
  ...
@mingyi456 mingyi456 added type.Task Something that needs to be done, but not a story, bug, or an epic. e.g. Move testing code into a new priority.High labels Oct 28, 2021
@mingyi456 mingyi456 added this to the v1.3 milestone Oct 28, 2021
@mingyi456 mingyi456 requested a review from wlren October 28, 2021 16:02
@codecov-commenter
Copy link

Codecov Report

Merging #101 (7c57984) into master (d5b1f61) will decrease coverage by 13.75%.
The diff coverage is 48.12%.

Impacted file tree graph

@@              Coverage Diff              @@
##             master     #101       +/-   ##
=============================================
- Coverage     72.15%   58.40%   -13.76%     
- Complexity      399      591      +192     
=============================================
  Files            70      120       +50     
  Lines          1232     2505     +1273     
  Branches        125      240      +115     
=============================================
+ Hits            889     1463      +574     
- Misses          311      953      +642     
- Partials         32       89       +57     
Impacted Files Coverage Δ
src/main/java/seedu/address/MainApp.java 0.00% <0.00%> (ø)
...main/java/seedu/address/commons/core/Messages.java 0.00% <ø> (ø)
...in/java/seedu/address/commons/util/StringUtil.java 64.28% <0.00%> (-30.46%) ⬇️
.../java/seedu/address/logic/commands/AddCommand.java 85.71% <0.00%> (-14.29%) ⬇️
...java/seedu/address/logic/commands/ExitCommand.java 100.00% <ø> (ø)
.../address/logic/commands/task/PurgeTaskCommand.java 0.00% <0.00%> (ø)
.../java/seedu/address/logic/parser/ArgumentType.java 0.00% <0.00%> (ø)
...eedu/address/logic/parser/DeleteCommandParser.java 100.00% <ø> (ø)
.../seedu/address/logic/parser/TaskCommandParser.java 0.00% <0.00%> (ø)
...ser/exceptions/ArgumentContainsSlashException.java 0.00% <0.00%> (ø)
... and 83 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d5b1f61...7c57984. Read the comment docs.

@wlren wlren merged commit 8536426 into AY2122S1-CS2103-F09-2:master Oct 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority.High type.Task Something that needs to be done, but not a story, bug, or an epic. e.g. Move testing code into a new
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants