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

PPP #168

Merged
merged 18 commits into from
Nov 9, 2020
Merged

PPP #168

merged 18 commits into from
Nov 9, 2020

Conversation

ChengJiyuqing
Copy link

No description provided.


Given below are my contributions to the project.

* **New Feature**: Added the ability to undo/redo previous commands.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you forgot to edit this part since we don't have an undo/redo command

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okay changed!

* Highlights: This enhancement affects existing commands and commands to be added in future. It required an in-depth analysis of design alternatives. The implementation too was challenging as it required changes to existing commands.
* Credits: *{mention here if you reused any code/ideas from elsewhere or if a third-party library is heavily used in the feature so that a reader can make a more accurate judgement of how much effort went into the feature}*

* **New Feature**: Added a history command that allows the user to navigate to previous commands using up/down keys.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here

@codecov-io
Copy link

Codecov Report

Merging #168 (c83e698) into master (6a30122) will decrease coverage by 0.18%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #168      +/-   ##
============================================
- Coverage     70.57%   70.39%   -0.19%     
- Complexity      481      488       +7     
============================================
  Files            79       79              
  Lines          1451     1459       +8     
  Branches        162      166       +4     
============================================
+ Hits           1024     1027       +3     
- Misses          377      380       +3     
- Partials         50       52       +2     
Impacted Files Coverage Δ Complexity Δ
...edu/medmoriser/logic/parser/QuizCommandParser.java 62.06% <0.00%> (-9.36%) 13.00% <0.00%> (+7.00%) ⬇️

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 6a30122...c83e698. Read the comment docs.

@ChengJiyuqing ChengJiyuqing merged commit fe884c8 into AY2021S1-CS2103T-W15-1:master Nov 9, 2020
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.

None yet

5 participants