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

[T16-2] SuperTA #62

Open
wants to merge 539 commits into
base: master
Choose a base branch
from

Conversation

yuheong
Copy link

@yuheong yuheong commented Sep 19, 2018

Copy link
Member

@tanhengyeow tanhengyeow left a comment

Choose a reason for hiding this comment

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

Good effort in updating the required documents! Great to see the comments made in tutorial reflected in the documents :) Please remember to update README.adoc page as per the requirements.

Please work towards the v1.1 milestone and update the documents according to any updates. Also, please do not close this PR as feedback would be made in this PR for subsequent milestones 😄

. Download the latest `addressbook.jar` link:{repoURL}/releases[here].
. Copy the file to the folder you want to use as the home folder for your Address Book.
. Download the latest `superTA.jar` link:{repoURL}/releases[here].
. Copy the file to the folder you want to use as the home folder for your SuperTA client.
. Double-click the file to start the app. The GUI should appear in a few seconds.
+
image::Ui.png[width="790"]
Copy link
Member

Choose a reason for hiding this comment

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

Might want to consider updating the picture to your mockup of the app.

Copy link
Member

@tanhengyeow tanhengyeow left a comment

Choose a reason for hiding this comment

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

Good job for v1.1! Some points to note:

  1. Please acknowledge AB4 under acknowledgements, something like AddressBook-Level4 project created by SE-EDU initiative at https://github.com/se-edu/
  2. Include features that are Not yet implemented as Coming in v2.0.

💪 💪 for the next milestone :)

Copy link
Member

@tanhengyeow tanhengyeow left a comment

Choose a reason for hiding this comment

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

Good work so far! As mentioned in tutorial, do prioritize the remaining tasks and break them down into manageable chunks for the rest of the milestones 😄

Ideally, all the major tasks should be done by v1.3 so the team can focus on fixing bugs, docs and handling minor updates after that.

bannified pushed a commit to bannified/addressbook-level4 that referenced this pull request Oct 21, 2018
…reposenseConfig

add config for reposense
pangjiahao pushed a commit to pangjiahao/cs2103-u-schedule that referenced this pull request Oct 28, 2018
Bellaaarh pushed a commit to Bellaaarh/addressbook-level4 that referenced this pull request Oct 29, 2018
…julian_changes

Add Calendar Events Synchronization / Test fixes
darieca and others added 30 commits November 12, 2018 12:51
update-assignment fix + SuperTaClientParserTests
View feedback bug, increase test coverage for view-feedback
Modified various Command messages for formatting
Final updates to fix PPP not rendering properly
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
7 participants