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.

Sign up for GitHub

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

Jump to bottom

Implement NoteFragment filtering feature #212

Merged
merged 22 commits into from Oct 29, 2019
Merged

Implement NoteFragment filtering feature #212

merged 22 commits into from Oct 29, 2019

Conversation

Sam-limyr
Copy link

Added:

  • NoteFragment class to represent portions of a Note's content.
  • Several ParserUtil methods for use of parsing NoteFragments from Notes.
  • Ability for NoteFragments to be seen in FilterAllByTag.
  • Some test cases for NoteFragment methods.
  • Support for two different types of Note viewing
  • Ability to view tags in the GUI
    Changed:
  • ViewNoteCommand has been changed to show a cleaned Note.
  • The old ViewNoteCommand is now renamed to ViewRawNoteCommand.

@Sam-limyr
Added a note in the DG.
371e5b9
@Sam-limyr
Merge branch 'merge-integration' of https://github.com/AY1920S1-CS210…
0373581 
…3T-W13-3/main
@Sam-limyr
Merge branch 'merge-integration' of https://github.com/AY1920S1-CS210…
d011526 
…3T-W13-3/main
@Sam-limyr
Merge branch 'merge-integration' of https://github.com/AY1920S1-CS210…
b49700c 
…3T-W13-3/main
@Sam-limyr
Add ability to view tags for ViewNoteCommand.
fc38f00
@Sam-limyr
Merge branch 'merge-integration' of https://github.com/AY1920S1-CS210…
c709dc0 
…3T-W13-3/main
@Sam-limyr
Create NoteFragment class to represent fragments of notes that are ta…
b2108ec 
…gged.
@Sam-limyr
Create methods to parse NoteFragments from Notes.
1be7fab
@Sam-limyr
Add test cases for newly added ParserUtil NoteFragment methods.
a16d39a
@Sam-limyr
Add placeholder list in Note to hold NoteFragments.
e3205ae
@Sam-limyr
Merge branch 'merge-integration' of https://github.com/AY1920S1-CS210…
e85f12e 
…3T-W13-3/main
@Sam-limyr
Allow NoteFragments to be viewed in filterall command.
51d056d
@Sam-limyr
Alter print format of Notes and NoteFragments to be in line with Flas…
ea368eb 
…hcards.
@Sam-limyr
Clean up logging messages in ParserUtil.
9b7b49e
@Sam-limyr
Change printing format of FilterAllByTag command.
30f0242
@Sam-limyr
Fork ViewNoteCommand as ViewRawNoteCommand.
28774a6
@Sam-limyr
Adapt ViewNoteCommand to show a cleaned Note.
5cb7e5c
@Sam-limyr
Merge branch 'merge-integration' of https://github.com/AY1920S1-CS210…
d5ba888 
…3T-W13-3/main
@Sam-limyr
Correct checkstyle errors.
fef51da
@Sam-limyr
Add Javadoc comments to new ParserUtil methods.
7c4a7d2
@Sam-limyr
Merge branch 'merge-integration' of https://github.com/AY1920S1-CS210…
e703170 
…3T-W13-3/main
@Sam-limyr Sam-limyr added this to the v1.3 milestone Oct 29, 2019
@Sam-limyr Sam-limyr self-assigned this Oct 29, 2019
@ckb055
Copy link

ckb055 commented Oct 29, 2019

Thank you!

@ckb055 ckb055 merged commit 1f81fb2 into AY1920S1-CS2103T-W13-3:merge-integration Oct 29, 2019
@nus-se-bot nus-se-bot mentioned this pull request Oct 29, 2019
Closed
27 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Sam-limyr Sam-limyr

Labels
None yet
Projects
None yet
Milestone
v1.3
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Sam-limyr @ckb055