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

Block function #40

Merged
merged 33 commits into from
Oct 20, 2016
Merged

Block function #40

merged 33 commits into from
Oct 20, 2016

Conversation

wengkiat
Copy link

@wengkiat wengkiat commented Oct 20, 2016

"I should also be able to view a list of dates and times that I have blocked, in case I forget."

Implementation

The block function uses an empty Task with Detail and no assigned Priority. These pseudotasks do not show up like normal tasks when listing or finding

  • ModifiedTask to include an extra parameter Blocking. These are by default false and are only toggled when the block command is called. No change to existing Task constructors.
  • Implement block command to toggle Task > Blocking. The command should take format

    block 10/20/2016 2pm to 10/21/2016 4pm block tmr to 10/20/2016 block noon to midnight

  • Block by default creates block time slot of one hour if only start time is entered, etc block 'meeting' 1300. The blocked time slot will be from 1300 to 1400
  • Before adding or editing tasks, the CheckBlocked class will run to determine if the input start/end date/time range conflicts with any blocked pseudotask. If so, an error message is thrown to the user.
  • Entering list block or lb would show a list of the pseudotasks.
  • Test cases created and passed for all the above implemented functions.

wengkiat and others added 30 commits October 19, 2016 16:36
Added message for block command
Created a boolean variable block to differinate between a task and a block time slot
To get updates from the main develop branch
Test for block command
edited message to fix test
@njyjn njyjn merged commit 0b4cda6 into v0.3 Oct 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants