Update to new posterous API #402

Closed
wants to merge 7 commits into
from

Conversation

Projects
None yet
5 participants
Contributor

alagu commented Sep 16, 2011

Posterous (http://posterous.com/api) changed their API and here is the code to support it.

This has to be updated in https://github.com/mojombo/jekyll/wiki/blog-migrations

simensen referenced this pull request in alagu/jekyll Nov 26, 2011

Merged

Fixed critical bug, added a few minor features #1

Contributor

simensen commented Nov 26, 2011

This is great! I tested it out and it was pretty close to working. I made a couple of tweaks and hope to have my changes added to this PR so that this updated Posterous importer will work better and also include the ability to capture tags from Posterous as well.

@alagu alagu Merge pull request #1 from simensen/alagu-master
Fixed critical bug, added a few minor features
20385b5
Contributor

alagu commented Jan 4, 2012

@mojombo can you please pull this patch?

Contributor

mojombo commented Jan 23, 2012

I just merged #472 which has caused this PR to conflict. Would you mind taking a look at the conflicts and seeing if you can rework this PR into a mergeable state? I'll be happy to merge it in once it's ready.

Contributor

simensen commented Jan 23, 2012

@alagu Let me know if you want me to take a crack at it. :)

Contributor

alagu commented Jan 23, 2012

@simensen Sure, it would be great if you could have a look at it :)

simensen added some commits Jan 23, 2012

@simensen simensen Merge remote-tracking branch 'origin/alagu-master' into alagu-postero…
…us-importer-rebased

Conflict created due to #472.

Conflicts:
	lib/jekyll/migrators/posterous.rb
5d6e87b
@simensen simensen Handle Posterous rate limiting 0284b8b
Contributor

simensen commented Jan 23, 2012

@mojombo I created a new PR (#477) with these patches + merge + bonus rate limit handling. The rate limiting issue was discovered while I was testing my conflict resolution skills. :)

@alagu Thanks again for starting this one. :)

Contributor

alagu commented Jan 25, 2012

@simensen Thanks :)

is this ready for merge? If it is, can we get this merged in? I'd like to use the migrator, but if it needs some work to be ready, I can try my hand at managing the conflicts if it is still in some limbo.

Contributor

alagu commented Jun 14, 2012

I am not sure about it though. I used it quite long time ago, and @simensen did couple of fixes. Did you try it out if it is working? If not I can look into it.

I'll take a look and see if it works.

Contributor

simensen commented Jun 14, 2012

See my other pr listed earlier. I think I recall thinking this pr
should be closed in favor of that one.

Sent from my iPhone

On Jun 14, 2012, at 6:04 PM, Jed Schneider
reply@reply.github.com
wrote:

I'll take a look and see if it works.


Reply to this email directly or view it on GitHub:
mojombo#402 (comment)

alagu closed this Oct 4, 2012

alagu referenced this pull request Oct 4, 2012

Merged

Posterous importer update #651

jekyllbot locked and limited conversation to collaborators Feb 27, 2017

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