Skip to content

Added scheduled Date query#48

Open
StephenOTT wants to merge 1 commit intoseomoz:masterfrom
StephenOTT:master
Open

Added scheduled Date query#48
StephenOTT wants to merge 1 commit intoseomoz:masterfrom
StephenOTT:master

Conversation

@StephenOTT
Copy link

for getting the score of the scheduled JID. Related to seomoz/qless#60

 for getting the score of the scheduled JID
@dlecocq
Copy link

dlecocq commented May 19, 2014

Looks good to me, but the tests need some fixing. In particular, making sure the tests know about the new 'scheduled' attribute.

@bkirz
Copy link
Contributor

bkirz commented May 19, 2014

Isn't calling this a date incorrect? I thought we were storing a timestamp. What about scheduled_for?

@StephenOTT
Copy link
Author

Isn't calling this a date incorrect? I thought we were storing a timestamp. What about scheduled_for?

agree. There is some cleanup to do. Can make the changes on the next pass with fixing up the tests.

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.

3 participants