Mark todo as expired feature #75

Closed
dranzd opened this Issue Mar 18, 2016 · 4 comments

Comments

Projects
None yet
2 participants
@dranzd
Contributor

dranzd commented Mar 18, 2016

What's the requirement for this feature?

Is this a feature where user sends a command to mark as expired or is this marked by the system when it goes expired?

Does expired means that the current time is past that deadline?

@codeliner

This comment has been minimized.

Show comment
Hide comment
@codeliner

codeliner Mar 18, 2016

Member

Hey @dranzd , do you want to take over the task? Yes, the system should mark a todo as expired when time has passed the deadline set by the user.

Similar to #73 a script is needed with a worker checking all open todos with a deadline and if one deadline has gone past a MarkTodoAsExpired command should be dispatched by the worker.
Let me know if you need more information.

Member

codeliner commented Mar 18, 2016

Hey @dranzd , do you want to take over the task? Yes, the system should mark a todo as expired when time has passed the deadline set by the user.

Similar to #73 a script is needed with a worker checking all open todos with a deadline and if one deadline has gone past a MarkTodoAsExpired command should be dispatched by the worker.
Let me know if you need more information.

@dranzd

This comment has been minimized.

Show comment
Hide comment
@dranzd

dranzd Mar 18, 2016

Contributor

@codeliner yes I want to try it out. I've already started that and most of the questions above are just to confirm that I understand what the feature does.

Contributor

dranzd commented Mar 18, 2016

@codeliner yes I want to try it out. I've already started that and most of the questions above are just to confirm that I understand what the feature does.

@codeliner

This comment has been minimized.

Show comment
Hide comment
@codeliner

codeliner Mar 18, 2016

Member

cool, looking forward to your PR

Member

codeliner commented Mar 18, 2016

cool, looking forward to your PR

@codeliner

This comment has been minimized.

Show comment
Hide comment
@codeliner

codeliner Apr 22, 2016

Member

PR merged #76 . Works like a charm.

Member

codeliner commented Apr 22, 2016

PR merged #76 . Works like a charm.

@codeliner codeliner closed this Apr 22, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment