Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

get_orders_by_date doesn't parse Date #6

Closed
mikelarkin opened this Issue Mar 13, 2012 · 2 comments

Comments

Projects
None yet
2 participants
Contributor

mikelarkin commented Mar 13, 2012

The parameter for the method is called "date", but it's actually expecting an already formatted RFC 2822 datetime string.

I'll fork and fix.

Contributor

mikelarkin commented Mar 13, 2012

invoking get_orders_by_date(Date.today) raises:

NoMethodError: private method `gsub' called for #<Date: 4911999/2,0,2299161>

@mikelarkin mikelarkin referenced this issue Mar 14, 2012

Merged

Orders by date #7

Contributor

mikelarkin commented Mar 14, 2012

Submitted Pull Request, thanks!

@mikelarkin mikelarkin closed this Jul 15, 2013

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