[T7][F11-B2] #44

levyjr · 2017-03-04T10:40:28Z

V0.0

xpdavid

You must be clear what you are going to build at the end. Basically, it's like implement the user guide step by step. Thus, it is better (required) to complete the user guide at the beginning.

xpdavid · 2017-03-05T18:09:00Z

README.md

@@ -1,36 +1,25 @@
-# Address Book (Level 4)
+# main


Maybe you want to change main to your app name. :P

xpdavid · 2017-03-05T18:09:44Z

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)
-[![Codacy Badge](https://api.codacy.com/project/badge/Grade/fc0b7775cf7f4fdeaf08776f3d8e364a)](https://www.codacy.com/app/damith/addressbook-level4?utm_source=github.com&amp;utm_medium=referral&amp;utm_content=se-edu/addressbook-level4&amp;utm_campaign=Badge_Grade)
+[![Build Status](https://travis-ci.org/CS2103JAN2017-F11-B2/main.svg?branch=master)](https://travis-ci.org/CS2103JAN2017-F11-B2/main)
+[![Build status](https://ci.appveyor.com/api/projects/status/3boko2x2vr5cc3w2?svg=true)](https://ci.appveyor.com/project/damithc/main)


This is not the correct update for appveyor.

BTW, if you don't want appveyor, you can just delete it.

xpdavid · 2017-03-05T18:10:27Z

README.md

+[![Build Status](https://travis-ci.org/CS2103JAN2017-F11-B2/main.svg?branch=master)](https://travis-ci.org/CS2103JAN2017-F11-B2/main)
+[![Build status](https://ci.appveyor.com/api/projects/status/3boko2x2vr5cc3w2?svg=true)](https://ci.appveyor.com/project/damithc/main)
+[![Coverage Status](https://coveralls.io/repos/github/CS2103JAN2017-F11-B2/main/badge.svg?branch=master)](https://coveralls.io/github/CS2103JAN2017-F11-B2/main?branch=master)
+[![Codacy Badge](https://api.codacy.com/project/badge/Grade/fc0b7775cf7f4fdeaf08776f3d8e364a)](https://www.codacy.com/app/damith/main?utm_source=github.com&amp;utm_medium=referral&amp;utm_content=CS2103JAN2017-F11-B2/main&amp;utm_campaign=Badge_Grade)


This is not the correct update.
Refer to the Codacy badge for more information.

xpdavid · 2017-03-05T18:11:37Z

README.md


 #### Site Map
 * [User Guide](docs/UserGuide.md)
 * [Developer Guide](docs/DeveloperGuide.md)
 * [Learning Outcomes](docs/LearningOutcomes.md)
-* [About Us](docs/AboutUs.md)


You are supposed to give About Us page. Refer to the module website for more information.

xpdavid · 2017-03-05T18:12:03Z

README.md


-<img src="docs/images/Ui.png" width="600"><br>


Do you have UI prototype or mockup?

xpdavid · 2017-03-05T18:15:59Z

docs/UserGuide.md

@@ -1,186 +1,85 @@
-# AddressBook Level 4 - User Guide
+# User Guide


Make sure you fulfill all the must-to-have features. :)

xpdavid · 2017-03-05T18:18:32Z

docs/UserGuide.md

+
+## Adding a task : `add`
+Adds a task to the task manager<br>
+Format: `add TASKNAME DATE [STARTTIME] [ENDTIME] VENUE DESCRIPTION` 


Think about the user experience. Does user need to remember the order of parameters? How the user will know which one is STARTTIME, which one is ENDTIME and etc.

xpdavid · 2017-03-05T18:19:34Z

docs/UserGuide.md

-### 2.6. Deleting a person : `delete`
+## Deleting a task : `delete`
+Deletes a specified tasks from the task manager. Irreversible.<br>
+Format: `delete TASKNAME`


Consider: Is the TASKNAME exact match or keyword match. How are you going to do if there are multiple tasks founded. Delete them all?

xpdavid · 2017-03-05T18:19:57Z

docs/UserGuide.md

-### 2.7. Select a person : `select`
+## View description of a task : `view`
+Displays details of the specified task.<br>
+Format: `view TASKNAME`


Same here. Multiple matched tasks?

xpdavid · 2017-03-05T18:20:39Z

docs/UserGuide.md

-Format: `clear`
-
-### 2.9. Exiting the program : `exit`
+## Clearing all entries : `delete`


Type here delete. Should be clear?

xpdavid

For developer guide. :)
Good work, Keep it up!

xpdavid · 2017-03-06T12:53:10Z

docs/DeveloperGuide.md

@@ -1,6 +1,6 @@
-# AddressBook Level 4 - Developer Guide


Remember to update the Design when you make changes to the code base.
You don't want to push the documentations thing in the end right?

xpdavid · 2017-03-06T12:53:58Z

docs/DeveloperGuide.md

-`* * *` | user | find a person by name | locate details of persons without having to go through the entire list
-`* *` | user | hide [private contact details](#private-contact-detail) by default | minimize chance of someone else seeing them by accident
-`*` | user with many persons in the address book | sort persons by name | locate a person easily
+`* * *` | user | add a new task|


Impressive features :)
But remember that you don't have enough time to implement every features, better prioritise them and schedule them in the milestones.

xpdavid · 2017-03-06T12:54:06Z

docs/DeveloperGuide.md


-#### Use case: Delete person


Good to see you have use cases.
Remember you can follow the interaction in use cases to implement the feature (as actually this how use cases are supposed to work).

Updated Aboutus, developer guide and learningoutcomes

# Conflicts: # collated/main/A0146757R.md # collated/main/A0152855Y.md # collated/main/A0163845X.md # collated/main/A0164061N.md # collated/test/A0146757R.md # collated/test/A0164061N.md

This reverts commit 2c681d4.

This reverts commit 46f2f32.

Yunchun

nus-se-pr-bot assigned cmkumar87 Mar 4, 2017

xpdavid reviewed Mar 5, 2017

View reviewed changes

xpdavid added the Reviewed label Mar 5, 2017

xpdavid assigned xpdavid and unassigned cmkumar87 Mar 5, 2017

xpdavid reviewed Mar 6, 2017

View reviewed changes

alvinlyj added a commit to KohJunWei/ProcrastiNoMore that referenced this pull request Mar 10, 2017

Merge pull request nus-cs2103-AY1617S2#44 from alvinlyj/V0.2

05eefad

Updated Aboutus, developer guide and learningoutcomes

edwinyeap and others added 23 commits April 7, 2017 01:38

fixed trailing whitespace

ba3232e

fixed trailing whitespace

19bdde4

fixed trailing whitespace

32a4a30

Update UserGuide.md

390fa4c

fixed indentation issues

fdbe868

Merge branch 'master' of https://github.com/CS2103JAN2017-F11-B2/main

2a6a5c7

fixed indentation issues

68ec522

fixed indentation issues

78f1751

fixed indentation issues

e7d91f8

fixed indentation issues

064dfac

fixed indentation issues

bc1540f

fixed indentation issues

e086e43

fixed indentation issues

f087f42

fixed filetab checker issues

2574b44

fixed filetab checker issues

a7481b9

fixed filetab checker issues

7c6405c

minor fix

19be32a

removed unsued imports

7be11b0

fixing order of imports

609bca3

fix import orders

d968e3c

fix import orders

fe7328e

fix import order issues

68657aa

fix import issues

7cb7ecb

yunchun-june and others added 30 commits April 9, 2017 23:04

travis

8e864ee

import errors

67bfa66

no message

ba43a43

no message

29be2d0

finished authoring code

fa7876f

Merge branch 'master' of https://github.com/CS2103JAN2017-F11-B2/main

7e7d3fc

Merge branch 'master' of ssh://github.com/CS2103JAN2017-F11-B2/main

7d923bd

Delete A0146757R.md

a3e2df8

Delete A0152855Y.md

9c1e657

Delete A0163845X.md

3a43eaa

Delete A0164061N.md

93b885c

Delete A0146757R.md

ec954d0

new commits

ac09978

Delete A0164061N.md

8b28361

Merge branch 'master' of https://github.com/CS2103JAN2017-F11-B2/main

d8bfe59

Add Final Collated Files

dd26a81

Merge branch 'master' of ssh://github.com/CS2103JAN2017-F11-B2/main

1d056ef

# Conflicts: # collated/main/A0146757R.md # collated/main/A0152855Y.md # collated/main/A0163845X.md # collated/main/A0164061N.md # collated/test/A0146757R.md # collated/test/A0164061N.md

Update AboutUs.md

72400ec

fixed trailing whitespace

723521c

Update README.md

46f2f32

Update UserGuide.md

2c681d4

Revert "Update UserGuide.md"

f8b4e76

This reverts commit 2c681d4.

Revert "Update README.md"

5c5046e

This reverts commit 46f2f32.

Possible Showcase Command Test

d54b8ec

Possible Showcase Command Test

e45b80b

trailing white space

55750e9

bug fix

b4c0922

Merge branch 'master' into yunchun

e1cf073

Merge pull request #52 from CS2103JAN2017-F11-B2/yunchun

b10c932

Yunchun

bug fix

05e7113

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[T7][F11-B2] #44

[T7][F11-B2] #44

levyjr commented Mar 4, 2017

xpdavid left a comment

xpdavid Mar 5, 2017

xpdavid Mar 5, 2017

xpdavid Mar 5, 2017

xpdavid Mar 5, 2017

xpdavid Mar 5, 2017

xpdavid Mar 5, 2017

xpdavid Mar 5, 2017

xpdavid Mar 5, 2017

xpdavid Mar 5, 2017

xpdavid Mar 5, 2017

xpdavid left a comment

xpdavid Mar 6, 2017

xpdavid Mar 6, 2017

xpdavid Mar 6, 2017

		@@ -1,186 +1,85 @@
		# AddressBook Level 4 - User Guide
		# User Guide

[T7][F11-B2] #44

Are you sure you want to change the base?

[T7][F11-B2] #44

Conversation

levyjr commented Mar 4, 2017

xpdavid left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

xpdavid left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment