[T7][T11-C2] by nahata-shailesh · Pull Request #68 · nus-cs2103-AY1617S1/addressbook-level4

nahata-shailesh · 2016-10-05T18:45:55Z

No description provided.

xdshang

@nahata-shailesh @Hastyrush @e0003083 @cheo1994 Some comments added. Please close the PR after reading comments.

xdshang · 2016-10-08T14:53:07Z

README.md

 [![Coverage Status](https://coveralls.io/repos/github/se-edu/addressbook-level4/badge.svg?branch=master)](https://coveralls.io/github/se-edu/addressbook-level4?branch=master)

-# Address Book (Level 4)
+[![Build Status](https://travis-ci.org/se-edu/addressbook-level4.svg?branch=master)](https://travis-ci.org/se-edu/addressbook-level4)


Duplicated stuff here.

xdshang · 2016-10-08T14:56:05Z

README.md

-    * A more sophisticated GUI that includes a list panel and an in-built Browser.
-    * More test cases, including automated GUI testing.
-    * Support for *Build Automation* using Gradle and for *Continuous Integration* using Travis CI.
+* Our product is a to-do list that accepts natural language commands via keyboard. The product is specifically 	catered towards users who   spend prolonged durations in front of the computer and prefer using solely the keyboard. On top of all these, the product is fully         functional offline with a variety of commands provided in the user guide which are easily understood.


Avoid lines longer than 110 chars. When a line is too long, it requires horizontal scrolling, that slows down the reader/reviewer. Both of docs and source codes should meet this requirement.

xdshang · 2016-10-08T14:57:59Z

README.md

+* Our product is a to-do list that accepts natural language commands via keyboard. The product is specifically 	catered towards users who   spend prolonged durations in front of the computer and prefer using solely the keyboard. On top of all these, the product is fully         functional offline with a variety of commands provided in the user guide which are easily understood.


 #### Site Map


Should acknowledge the original source of the code, i.e. the SE-EDU initiative at https://github.com/se-edu/

xdshang · 2016-10-08T15:02:21Z

docs/AboutUs.md

-<img src="images/LeowYijin.jpg" width="150"><br>
-Role: Developer <br>  
-Responsibilities: Data
+#### [Tan Jun Hao](https://github.com/e0003083)


Please provide a photo

xdshang · 2016-10-08T16:55:45Z

docs/UserGuide.md

+	(if you type this on a Wednesday, it marks the slot not for the next day, but for the Thursday after that, that is, 8 days later)
+* `add EE2020 project by 30/10/2016 at 6pm`
+
+#### Priority tasks: `priority`


The description of this feature should be merged into the description of add command, because it is not a complete command.

xdshang · 2016-10-08T16:57:07Z

docs/UserGuide.md

+Format: `delete <task index>`
+
+Example:
+* `delete s1`


Please provide more information about the format of <task index>. What does s1 mean here?

xdshang · 2016-10-08T16:58:55Z

docs/UserGuide.md

-**A**: Install the app in the other computer and overwrite the empty data file it creates with 
-       the file that contains the data of your previous Address Book folder.
-
-## Command Summary


Better to keep this command summary, because it is easy for users to find what commands your app has.

xdshang · 2016-10-08T17:04:11Z

docs/UserGuide.md

+Example: 
+* `add-s t2 Question1`
+
+##### Set alarms for tasks: `remind`


How do you prepare to alarm users? Raise a system notification, or send an email?

xdshang · 2016-10-08T17:08:21Z

docs/UserGuide.md

+Example:
+* `delete s1`
+
+#### View: `donelist`


This feature can be merged with the feature home, by using a consolidated command, say show <content>. Then you can add more options, like show <date> to display tasks on a specific day.

xdshang · 2016-10-08T17:15:37Z

docs/UserGuide.md

+* `done t1`
+
+#### Adding a subtask: `add-s`
+Adds a subtask to a main task<br>


Maybe this is a subfeature of edit command. I think you are just updating a specific task by adding some content.

…CS2103AUG2016-T11-C2/main into sn-add-autocomplete-and-history # Conflicts: # src/main/java/seedu/forgetmenot/model/TaskManager.java

…CS2103AUG2016-T11-C2/main into sn-add-autocomplete-and-history # Conflicts: # src/main/java/seedu/forgetmenot/ui/ContentBox.java

…CS2103AUG2016-T11-C2/main into sn-add-autocomplete-and-history * 'sn-add-autocomplete-and-history' of https://github.com/CS2103AUG2016-T11-C2/main: add cleanup to UI parts and collate of qy code add test for done and undone Fix buid gradle error SN-Added author tags add @@author to test add @@author no message Add cleaning up to UI codes

added CY's part of the aboutus

Update AboutUs.md

…1-C2/main into sn-add-autocomplete-and-history

…-and-history Add QY author tags

…into sn-add-autocomplete-and-history

…-and-history Add tests

…-and-history Final author tags

…into sn-add-autocomplete-and-history

…-and-history Fix Initialisation bug

…-and-history Add overview

…-and-history Fix scroll bug

…-and-history Add collated files

…into sn-add-autocomplete-and-history

…-and-history Added Sample Data and Test Script

nus-se-pr-bot assigned xdshang Oct 5, 2016

xdshang reviewed Oct 8, 2016

View reviewed changes

xdshang added the Reviewed label Oct 8, 2016

nahata-shailesh force-pushed the master branch from c169b55 to 10f2b41 Compare October 14, 2016 04:34

nahata-shailesh force-pushed the master branch from d69de66 to 01281e9 Compare October 26, 2016 12:51

muahcheeee and others added 25 commits November 2, 2016 15:26

rebuilt gradle

1cc17f8

added author tags

64f0b05

no message

956af2f

add @@author

fe481af

Merge branch 'sn-add-autocomplete-and-history' of https://github.com/…

d82a845

…CS2103AUG2016-T11-C2/main into sn-add-autocomplete-and-history # Conflicts: # src/main/java/seedu/forgetmenot/model/TaskManager.java

add @@author to test

81801f5

SN-Added author tags

06115cd

update ug

4649713

Fix merge conflict

4276058

update @@author for ug

31d157a

Fix buid gradle error

cc1cc31

update ug

5ace59c

update about us

ba27f89

update AboutUs

86d7a91

update sn-aboutUs

e8fb2bb

Fix minor typo in AboutUs

fbf6299

add test for done and undone

249f4ae

add Counter test

a139d51

Merge branch 'sn-add-autocomplete-and-history' of https://github.com/…

c01d645

…CS2103AUG2016-T11-C2/main into sn-add-autocomplete-and-history # Conflicts: # src/main/java/seedu/forgetmenot/ui/ContentBox.java

add cleanup to UI parts and collate of qy code

09d9473

Update AboutUs.md

3088daa

reorganised toString in Task

b1d8747

Update AboutUs.md

8937818

added CY's part of the aboutus

Revert to old toString()

65f68e0

nahata-shailesh and others added 30 commits November 7, 2016 14:28

Merge pull request #108 from CS2103AUG2016-T11-C2/Hastyrush-patch-5

7bbb275

Update AboutUs.md

add collate for qy aboutus and DG

f52752d

Update readme

1442686

Add 1 author tag

1e06d44

Merge branch 'qy-collate-docs' of https://github.com/CS2103AUG2016-T1…

2e7eed9

…1-C2/main into sn-add-autocomplete-and-history

Merge pull request #111 from CS2103AUG2016-T11-C2/sn-add-autocomplete…

27a4733

…-and-history Add QY author tags

minor tweaks

2c7f2b2

Add commandHistory and AutoComplete Test

9a438e6

Merge branch 'master' of https://github.com/CS2103AUG2016-T11-C2/main …

f495a2a

…into sn-add-autocomplete-and-history

Merge pull request #112 from CS2103AUG2016-T11-C2/sn-add-autocomplete…

4cf5ba8

…-and-history Add tests

Final author tags

c3c64bf

Merge pull request #113 from CS2103AUG2016-T11-C2/sn-add-autocomplete…

ade164e

…-and-history Final author tags

minor changes

a08c4ea

Typo

1ed4010

Minor fix to qy collate

2b2cd4c

Fix initialisation bug

8784562

Merge branch 'master' of https://github.com/CS2103AUG2016-T11-C2/main …

48f7904

…into sn-add-autocomplete-and-history

Merge pull request #114 from CS2103AUG2016-T11-C2/sn-add-autocomplete…

1b1505d

…-and-history Fix Initialisation bug

Add overview

aee28ea

Merge pull request #115 from CS2103AUG2016-T11-C2/sn-add-autocomplete…

bf0fb6a

…-and-history Add overview

Minor fix

ff7b077

changed componenet to component

a0fccc1

Fix scroll bug

878b130

Merge pull request #116 from CS2103AUG2016-T11-C2/sn-add-autocomplete…

4a0e0cb

…-and-history Fix scroll bug

Add collated files

05bbb3e

Merge pull request #117 from CS2103AUG2016-T11-C2/sn-add-autocomplete…

c675eb1

…-and-history Add collated files

add edits to qy aboutus

507dc1e

Added test file and sample data

ee6ae88

Merge branch 'master' of https://github.com/CS2103AUG2016-T11-C2/main …

330d70d

…into sn-add-autocomplete-and-history

Merge pull request #118 from CS2103AUG2016-T11-C2/sn-add-autocomplete…

eaef270

…-and-history Added Sample Data and Test Script

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[T7][T11-C2]#68

[T7][T11-C2]#68
nahata-shailesh wants to merge 643 commits intonus-cs2103-AY1617S1:masterfrom
CS2103AUG2016-T11-C2:master

nahata-shailesh commented Oct 5, 2016

Uh oh!

xdshang left a comment

Uh oh!

xdshang Oct 8, 2016

Uh oh!

xdshang Oct 8, 2016

Uh oh!

xdshang Oct 8, 2016

Uh oh!

xdshang Oct 8, 2016 •

edited

Loading

Uh oh!

xdshang Oct 8, 2016

Uh oh!

xdshang Oct 8, 2016

Uh oh!

xdshang Oct 8, 2016

Uh oh!

xdshang Oct 8, 2016

Uh oh!

xdshang Oct 8, 2016

Uh oh!

xdshang Oct 8, 2016

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

		* Our product is a to-do list that accepts natural language commands via keyboard. The product is specifically catered towards users who spend prolonged durations in front of the computer and prefer using solely the keyboard. On top of all these, the product is fully functional offline with a variety of commands provided in the user guide which are easily understood.


		#### Site Map

Conversation

nahata-shailesh commented Oct 5, 2016

Uh oh!

xdshang left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

xdshang Oct 8, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

xdshang Oct 8, 2016 •

edited

Loading