-
Notifications
You must be signed in to change notification settings - Fork 231
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
[CS2113-T12-4] Omnitravel #36
base: master
Are you sure you want to change the base?
Changes from all commits
59523c6
0888f30
d013733
7287ea4
264d166
9e97d0e
c4ef836
6de20bd
1c117e6
a163433
3d46a95
97e873f
e72019c
5046e1a
7e8d9de
f62593a
d94bc34
672fce8
e852cb0
f9f62f5
0bf9ff1
cb17222
d16dc27
2773768
9865b47
f01b15c
c834996
20bfc8c
7381df2
67c407d
8b97fa2
a9824ae
125a4c7
fda1646
8339e0e
7ce04f8
adcb3ad
f391b2c
8634ab4
891f521
3fa653f
c9d7983
ffe1db2
9752e83
b30f4ec
61322b6
4739b2d
fdeb1ef
a70f890
c3e02d8
f774ac0
80a39a7
72aab9d
f6333ab
be60e42
00aee27
cfa7cb4
5213a22
0b2f8cb
b33ae42
7474b7b
57f25ec
137b914
b72f5c7
7d57396
404213c
37b69dc
b3fa1ca
7af7d40
75a98ae
67a2b96
4fc6850
396338b
d25ae04
657ecf1
162066f
451638c
31211b4
d23e6dd
00ad20c
d244686
a9c3d3c
c1b71d9
af42f4b
75b06b9
4ad8705
cd4071c
5f68f0b
0946029
289b16a
482d168
7b2a641
4bc2e5a
318cad0
d2de866
a5c75ec
03ebc12
2c67e13
d86ef9e
4a4c025
7e3736f
4d1ddde
9eb038c
cbbdbdc
4db31ef
66af77e
49223bc
f27556a
28014b7
d77928a
89ebba8
0ea8c89
314b1d4
5cfac12
4e14c8d
de29041
8fb864d
2b3d8e8
7900ac4
12e9154
8e1c4a2
f21e0c3
04a882d
0c916e6
b6918cb
0590f1b
fe640b3
338ca75
8b6b2c1
398f682
fc60e2c
8ed4232
87aefa9
5b6bd74
50c149c
f5df6bb
27d872d
12cba98
6ac7a69
584a8b4
2228fad
9c977c5
a7c8fcd
bb0515c
3dc286e
be80b9a
d765a97
effafc3
859d4cb
20fc87e
8081f1a
8035987
9263ecd
2babfeb
bd5bfe6
1bd0866
fc32865
81de061
32e096a
326bc6e
ad66e9e
254f0b2
e8fe319
6f8bdf7
b7776f7
8232670
3c6efff
152cedd
1aacdb6
db0314e
a831c1b
9155347
6154d6e
5d538c7
96ad643
f10b00c
e1fff42
32ae783
2e32714
9ca74e9
48c9b7c
92556fc
6c6c20b
2736530
da07f1e
538e94a
d6dfa93
3eb74f3
5773b2b
2a04ae1
0c649c7
afaac8a
19fa341
477e472
4ddbac3
4c8b21a
5c3e699
acd67b1
0c4d1db
177791c
8f32fd1
641cb7f
40c4d77
f188890
fe4b21c
13fe4c3
9cd8379
6c80ab5
83dd6ff
686eec4
2a00711
4bfa388
02c8480
b20066d
35d943f
ef6651c
fb04fa8
498d3b6
b10bc73
8bc1f41
34e4f54
a63881f
b6e2615
771ed2a
17a8aed
f86bd6d
f697891
4788b25
cd09c70
3ce1ead
683bb3c
bcd1ff7
fafc681
d7489c5
6470f7b
5c924d3
5536ba8
31bc348
5795206
ccf62ac
f1796d3
6d1f6a0
e5c8e80
32d14b2
844a6ea
a1e5e4f
db7febb
7a45d8b
a702047
d0161ec
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -15,3 +15,4 @@ bin/ | |
|
||
/text-ui-test/ACTUAL.TXT | ||
text-ui-test/EXPECTED-UNIX.TXT | ||
text-ui-test/ACTUAL-UNIX.TXT |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
Manifest-Version: 1.0 | ||
Main-Class: seedu.omnitravel.omnitravel.OmniTravel | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,9 +1,9 @@ | ||
# About us | ||
| Display | Name | Github Profile | Portfolio | | ||
|----------------------------------------------------:|:------------------:|:---------------------------------------------:|:-------------------------------------:| | ||
| ![](https://via.placeholder.com/100.png?text=Photo) | Eugene Chan Jiajun | [Github](https://github.com/EugeneChanJiajun) | [Portfolio](team/eugenechanjiajun.md) | | ||
| ![](https://via.placeholder.com/100.png?text=Photo) | Daryl Tay | [Github](https://github.com/daryltay415) | [Portfolio](team/daryltay415.md) | | ||
| ![](https://via.placeholder.com/100.png?text=Photo) | Annie Xu | [Github](https://github.com/annnniexu) | [Portfolio](team/annnniexu.md) | | ||
| ![](https://via.placeholder.com/100.png?text=Photo) | Chin Yan Xu | [Github](https://github.com/ChinYanXu) | [Portfolio](team/chinyanxu.md) | | ||
| ![](https://via.placeholder.com/100.png?text=Photo) | Chen Kang | [Github](https://github.com/ChenKangg) | [Portfolio](team/chenkangg.md) | | ||
|
||
Display | Name | Github Profile | Portfolio | ||
--------|:----:|:--------------:|:---------: | ||
![](https://via.placeholder.com/100.png?text=Photo) | John Doe | [Github](https://github.com/) | [Portfolio](docs/team/johndoe.md) | ||
![](https://via.placeholder.com/100.png?text=Photo) | Don Joe | [Github](https://github.com/) | [Portfolio](docs/team/johndoe.md) | ||
![](https://via.placeholder.com/100.png?text=Photo) | Ron John | [Github](https://github.com/) | [Portfolio](docs/team/johndoe.md) | ||
![](https://via.placeholder.com/100.png?text=Photo) | John Roe | [Github](https://github.com/) | [Portfolio](docs/team/johndoe.md) | ||
![](https://via.placeholder.com/100.png?text=Photo) | Don Roe | [Github](https://github.com/) | [Portfolio](docs/team/johndoe.md) |
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. very nice diagrams, but the contents of the diagram within the alt box is quite repetitive. However it is still very clear what is happening which is more important. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. For the parser diagram, maybe you could split the line updateTravelActivity(travelActivityNumber, date, duration) into two lines so that the gray parts of the diagram to the right aren't so far away. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. good usage of code and markdown to illustrate what is happening. Overall quite a clear DG, but maybe more examples of how to use each feature and expected output would be helpful. |
Large diffs are not rendered by default.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,9 @@ | ||
# Duke | ||
# Omnitravel | ||
|
||
{Give product intro here} | ||
|
||
Omnitravel is a CLI (Command-Line Interface) app that simplifies travel planning and organization. | ||
Users can add accommodations, food, and landmarks to their itinerary, manage activities easily, and track expenses. | ||
The app offers search, filtering, and currency exchange features, ensuring a smooth travel planning experience. | ||
|
||
Useful links: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Very detailed diagram of how the TravelActivityList works. |
||
* [User Guide](UserGuide.md) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this diagram a lot, very clear to see which classes implement and use which other classes. I like that you made this the first diagram in your TP as well, great work!