Skip to content

Comments

if post_date is within fifteen minutes of current time, use Medium time#113

Merged
huckphin merged 1 commit intomasterfrom
huckphin/fix-cross-post
Jul 28, 2016
Merged

if post_date is within fifteen minutes of current time, use Medium time#113
huckphin merged 1 commit intomasterfrom
huckphin/fix-cross-post

Conversation

@huckphin
Copy link
Contributor

Hello @amyquispe, @benmedium, @mikkot,

Please review the following commits I made in branch 'huckphin/fix-cross-post'.

e77628e (2016-07-28 13:50:07 -0700)
if post_date is within fifteen minutes of current time, use Medium time
There were reports of publishing with a cross-post failing. It turns out
that if you are even a second ahead of the Medium servers, the post will
fail to cross-post due to "posting in the future". If the post_date is
within 15 minutes of current time, remove the "publishedAt" attribute in
the POST body, and just use Medium server time for the actual post datetime.

Fixes #106

R=@amyquispe
R=@benmedium
R=@mikkot

There were reports of publishing with a cross-post failing. It turns out
that if you are even a second ahead of the Medium servers, the post will
fail to cross-post due to "posting in the future". If the post_date is
within 15 minutes of current time, remove the "publishedAt" attribute in
the POST body, and just use Medium server time for the actual post datetime.

Fixes #106
@amyquispe
Copy link
Contributor

lgtm

@huckphin huckphin merged commit e9ba881 into master Jul 28, 2016
@huckphin huckphin deleted the huckphin/fix-cross-post branch July 28, 2016 20:58
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.

Medium Error: If specified, publishedAt must be in the past. (2014)

2 participants