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

[T7][T14-C4] #72

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

Conversation

SukiTsang
Copy link

No description provided.

@hang15
Copy link

hang15 commented Oct 10, 2016

Some comments on your user guide:

Features/User experience

  1. Your datetime input seems to be very rigid or fixed. Try to accommodate/allow some degree of flexibility, eg. DD-MM-YY. What about an event with a fixed date and no fixed time?
  2. There are inconsistencies in your user guide on the datetime format. Please indicate if it is meant to be flexible.
  3. You may want to indicate/think through in detail how your undo feature should work. For eg. how many undos are allowed?

Housekeeping -
You need to acknowledge addressbook-level4 or its parent project/org for the code base.

Skaty and others added 27 commits November 3, 2016 10:16
Fix documentation and personal TA code issues
Just a simple fix for my code. No tests required, since it is a docstring.
Allow TaskList to be filtered through preset filters
Allow user to see the actual open and close time of tasks
Add link to DeveloperGuide and update mock-up picture on README
Skaty and others added 30 commits November 7, 2016 21:40
Remove open and close time tests as they are non-deterministic
Update with new commands in help page
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants