Automatically convert date and datetime to time or raise an exception #103

Closed
mhenrixon opened this Issue Oct 12, 2012 · 4 comments

Comments

Projects
None yet
2 participants
@mhenrixon

So right now to get the occurrences or any other such I have to for instance first convert a date to time. I would like ice_cube to either raise an exception or automatically convert the date to a time.

I can add this functionality myself but I want to know which path you would like to take. If you for instance only see ice_cube used in scenarios where there is a time then you would probably want the user to be told about this by raising an error.

If you however prefer the path of least resistance which might not be such a bad thing in case someone wants to know if something is scheduled today and just send a date in maybe that could be converted to time for these but not for scheduling.

Please report back with your thoughts so that I can start implementing this (we ran into some issues with not getting occurrences back because of sending date or datetime in)

@mhenrixon

This comment has been minimized.

Show comment Hide comment
@mhenrixon

mhenrixon Oct 12, 2012

2 scenarios come to think of it, scheduling something and retrieving something. Maybe those two should be treated differently?

2 scenarios come to think of it, scheduling something and retrieving something. Maybe those two should be treated differently?

@seejohnrun

This comment has been minimized.

Show comment Hide comment
@seejohnrun

seejohnrun Oct 13, 2012

Owner

So I have this working in a branch - still going to let it simmer for a few days to see how I feel about it

Owner

seejohnrun commented Oct 13, 2012

So I have this working in a branch - still going to let it simmer for a few days to see how I feel about it

@ghost ghost assigned seejohnrun Oct 13, 2012

@seejohnrun

This comment has been minimized.

Show comment Hide comment
@seejohnrun

seejohnrun Oct 19, 2012

Owner

Are people liking this feature?

Owner

seejohnrun commented Oct 19, 2012

Are people liking this feature?

@seejohnrun

This comment has been minimized.

Show comment Hide comment
@seejohnrun

seejohnrun Oct 28, 2012

Owner

This feature will go out with 0.9.2

Owner

seejohnrun commented Oct 28, 2012

This feature will go out with 0.9.2

seejohnrun added a commit that referenced this issue Oct 28, 2012

@seejohnrun seejohnrun closed this Oct 28, 2012

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