Update docs #149

INCENDE · 2016-10-29T02:35:48Z

Shifted the to-dos here. Fixes #104

Developer Guide

Compare with other alternative designs possible.
Add more descriptions e.g. show at least one interaction (i.e. sequence diagrams) within each component. But this doesn't mean to add more verbose text about trivial and unstable things.
Use object diagrams to show the object structure at runtime
Use activity diagrams and OO domain models where applicable (covered in yesterday's lecture)
Explain how they implemented things such as undo/redo (since they weren't given features in AddressBook Level 4)
explain other interesting design decisions you made.
sequence diagram for other command e.g. add?

- Make it more informal like 2101, directly address the user

- Alias supports multiple aliases - Unalias only support removing the alias command

…pdate-guides-for-v0.4 # Conflicts: # docs/DeveloperGuide.md # docs/UserGuide.md

rachx · 2016-11-07T09:47:32Z

Updated about us again (add more features and change the links to match the sub-headers of the new user guide))

rachx · 2016-11-07T10:17:13Z

Tick if you are okay with the files and ready to merge.

Rachael

User guide
Developer guide - think the justification for singleton/ add, schedule use cases should be updated but not changing it.

Weiguang

User guide
Developer guide

Justin

User guide
Developer guide

Vishnu

User guide
Developer guide

…2016-W11-C2/main into update-guides-for-v0.4

burnflare · 2016-11-07T12:47:53Z

src/main/java/seedu/agendum/ui/UiPart.java

@@ -74,7 +75,8 @@ protected Stage createDialogStage(String title, Stage parentStage, Scene scene)
     * @param iconSource e.g. {@code "/images/help_icon.png"}
     */
    protected void setIcon(String iconSource) {
-        primaryStage.getIcons().add(AppUtil.getImage(iconSource));
+//        primaryStage.getIcons().add(AppUtil.getImage(iconSource));
+        primaryStage.getIcons().add(new Image(this.getClass().getResourceAsStream(iconSource)));


Why is there commented code here?

Also, what does this code change?

O forgot to delete it !!!

Thanks! 👍

burnflare

To merge after addressing commented code

rachx · 2016-11-07T13:01:03Z

should we add sync interface in the diagram?

burnflare · 2016-11-07T13:04:55Z

@rachx To the Model diagram? Can you help me do it, I don't have PPT installed. 🎱

INCENDE and others added 13 commits October 25, 2016 11:43

Update sentence structures and wording

abbd158

- Make it more informal like 2101, directly address the user

Update spacing

3cc3597

Tag user and developer guide, update about us and collate

671cc7e

Update UserGuide.md

42f6e37

Update major contributions

6b53c62

Update matric no.

08ab635

Update matric no

3ce27fb

Add collate.bat

8539a63

Re-collate files

d40990b

Update dev guide tags

4ed2750

Re-collate

975434f

add missing bracket

1fc0751

Merge branch 'master' into update-about-us-tag-docs

c9fe030

rachx mentioned this pull request Oct 29, 2016

Update user and dev guide again #104

Closed

INCENDE added 15 commits October 29, 2016 10:36

Merge branch 'master' into update-guides-for-v0.4

3ff99d8

Update User stories

e88617b

Update NFR

ffb4504

Add images for user guide

10c92cc

Update quick start guide

ad1b652

Update features

a152302

Update UserGuide.md

3263aad

Change path to file for store and load

7339e2c

Update store and load use cases

19a5c49

Update user stories to match user guide

14837c4

Update alias and unalias

68fc3b0

- Alias supports multiple aliases - Unalias only support removing the alias command

Add unalias and unmark. Update alias

ac64263

Merge branch 'master' into update-guides-for-v0.4

92a8f28

Merge remote-tracking branch 'origin/update-about-us-tag-docs' into u…

2ca848f

…pdate-guides-for-v0.4 # Conflicts: # docs/DeveloperGuide.md # docs/UserGuide.md

Re-collate

008fc85

INCENDE mentioned this pull request Oct 29, 2016

Tag user and developer guide, update about us and collate #122

Closed

rachx added 2 commits November 7, 2016 17:44

update about us

c97fcce

remove collate

ac0785d

Fan Weiguang and others added 16 commits November 7, 2016 18:25

Update test script

da21b17

update diagrams ppt and storage diagram

db2a634

update test script

99e3392

Update acknowledgements

34b4fd5

Update ack

3f2367c

Update acknowledgements

073c03d

Update Acknowledgements (final)

80a6f5d

add google sync disclaimer

a85bb89

Merge branch 'update-guides-for-v0.4' of https://github.com/CS2103AUG…

7670390

…2016-W11-C2/main into update-guides-for-v0.4

Bring back UsingGradle.md from upstream project

fec44de

Fix Google Calendar anchor tag

475c8cc

Fix wording in Google Calendar section for accuracy

fa75590

minor grammar

6f76622

Add more dependencies

c16ea06

Add a paragarph about Sync

4e4b588

Tiny about-us update

476e037

burnflare reviewed Nov 7, 2016

View reviewed changes

Minor update to sync section

049ed14

burnflare approved these changes Nov 7, 2016

View reviewed changes

Delete some unusable comments

af6816f

rachx added 2 commits November 7, 2016 21:10

update model diagram

a87f2f3

merged remote

8339cd0

burnflare merged commit fb0f9c1 into master Nov 7, 2016

burnflare deleted the update-guides-for-v0.4 branch November 7, 2016 13:23

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update docs #149

Update docs #149

INCENDE commented Oct 29, 2016 •

edited

Loading

rachx commented Nov 7, 2016

rachx commented Nov 7, 2016 •

edited by burnflare

Loading

burnflare Nov 7, 2016

burnflare Nov 7, 2016

burnflare Nov 7, 2016

fanwgwg Nov 7, 2016

burnflare Nov 7, 2016

burnflare left a comment •

edited

Loading

rachx commented Nov 7, 2016

burnflare commented Nov 7, 2016

Update docs #149

Update docs #149

Conversation

INCENDE commented Oct 29, 2016 • edited Loading

rachx commented Nov 7, 2016

rachx commented Nov 7, 2016 • edited by burnflare Loading

burnflare Nov 7, 2016

Choose a reason for hiding this comment

burnflare Nov 7, 2016

Choose a reason for hiding this comment

burnflare Nov 7, 2016

Choose a reason for hiding this comment

fanwgwg Nov 7, 2016

Choose a reason for hiding this comment

burnflare Nov 7, 2016

Choose a reason for hiding this comment

burnflare left a comment • edited Loading

Choose a reason for hiding this comment

rachx commented Nov 7, 2016

burnflare commented Nov 7, 2016

INCENDE commented Oct 29, 2016 •

edited

Loading

rachx commented Nov 7, 2016 •

edited by burnflare

Loading

burnflare left a comment •

edited

Loading