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

Add TagCommand functionality #17

Merged
merged 14 commits into from Oct 1, 2019

Conversation

Kalsyc
Copy link

@Kalsyc Kalsyc commented Sep 30, 2019

Modifications: that will resolve #23.

  1. Add TagCommand functionality.
  2. Edit UserGuide to include TagCommand.
  3. Xpire now reads trimmed version of the firstWord when parsing.
    (i.e. add |banana|10/10/2019|#Fruit is valid)

To do:

  1. Add J-Unit Testing for TagCommand and TagCommandParser.
  2. Fix J-Unit Testing such that tag|1|Fruit#Food should only read #Food as the sole tag of the item at index 1 and not both #Fruit and #Food
  3. Add more documentation and methods (e.g. equals method) for defensive measures.

@febee99 febee99 added this to Review in progress in Xpire Oct 1, 2019
@JermyTan JermyTan merged commit c883558 into AY1920S1-CS2103T-F11-2:master Oct 1, 2019
Xpire automation moved this from Review in progress to Done Oct 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Xpire
  
Done
Development

Successfully merging this pull request may close these issues.

As a user, I can tag my items
2 participants