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

[CS2103T-T13-4] Mark #80

Open
wants to merge 1,061 commits into
base: master
Choose a base branch
from

Conversation

Na-Nazhou
Copy link

@Na-Nazhou Na-Nazhou commented Sep 24, 2019

Copy link

@rongrongrr rongrongrr left a comment

Choose a reason for hiding this comment

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

Hey! Great job, we see that you guys have started morphing your project. It's great to see that you are on track :)


1. User requests to list bookmarks
2. Mark shows a list of bookmarks
+

Choose a reason for hiding this comment

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

Extension for 2. e.g. if there are no bookmarks

alages97 pushed a commit to alages97/main that referenced this pull request Oct 26, 2019
openorclose and others added 25 commits November 8, 2019 09:47
Arranged commands into sections according to the type of data that they
operate on. Added an overview of Mark's layout in a top-down manner.
Amended the walk-through description for AddCommand and re-included
text details for other examples, as suggested by @kxyoke.
Update User guide to mark out areas that need screenshots
…ank. Logger message regarding internet connection is updated accordingly. Users will have to check themselves if their input was wrong or no internet connection
openorclose and others added 30 commits November 11, 2019 23:02
Unless screenshots need to be resized.
Update ppp to fit within 10 pages
# Conflicts:
#	docs/UserGuide.adoc
Fix: Typos and missing testing instructions
Fix delete all annotations bug: handle case when no cache is available.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants