Skip to content

Download and include images in posterous migrator.#4

Closed
parkr wants to merge 1 commit intoinitial-migrator-importfrom
mojombo/jekyll509
Closed

Download and include images in posterous migrator.#4
parkr wants to merge 1 commit intoinitial-migrator-importfrom
mojombo/jekyll509

Conversation

@parkr
Copy link
Member

@parkr parkr commented Jan 31, 2013

ACHTUNG!! This is being merged into a non-master branch.

In jekyll/jekyll#509, @markhepburn suggested that we import images from Posterous, and supplied the code. Thought it was a cool idea.

I haven't tested this (hadn't heard of Posterous before a couple weeks ago) but I think it worked for Mark.

Here's his original message:

I've added image importing to the posterous migrator; the other media types could be added similarly but I thought I'd push this out there first. (I'm not actually expecting this to go in as-is -- I'm new to both ruby and jekyll for a start! It also includes a few arbitrary decisions however)

It does the following:

  • downloads the "full" image from the media section of each post;
  • saves it in a directory imgs/;
  • removes the [[posterous-content:...]] block at the end of the content, and replaces it with an ordered list of images, including the base_path

(I figured a list should be easy to turn into a gallery, etc if people wanted to add their own js)

Is this useful?

Signed-off-by: Parker Moore <parkrmoore@gmail.com>
@parkr
Copy link
Member Author

parkr commented Jan 31, 2013

Having some way to turn this off when you call the importer would be way cool.

@markhepburn
Copy link
Contributor

Hey, thanks for taking a look at this. Like I said, it was just to get the ball rolling as much as anything else.

What do you think the best interface for disabling it would be? And opt-in or opt-out I suppose is the other question; maybe opt-in is preferable, since it will take more time and bandwidth. Perhaps a flag --include-images?

I think my posterous account is still active if you'd like me to play with it.

@parkr
Copy link
Member Author

parkr commented Jan 31, 2013

Hey! Thanks for getting back to me so quickly. I think the best interface would be to pass in an options has or something into the migrator that sets :include_images => true or something. We can encompass other options that way, too.

@parkr
Copy link
Member Author

parkr commented Jan 31, 2013

By default, though, I don't think I would want to download all those images.

@markhepburn
Copy link
Contributor

Sorry for the delay in getting back to you -- had to wait for the weekend :)

Thanks for the "keyword args" tip, like I said I haven't really done any ruby. That looks much nicer though. I've implemented that functionality anyway, and it works fine on my old account. Do you want me to issue another pull request? I've pushed the changes to my fork, although I haven't rebased against the current state of affairs.

Thanks for your work on this, it'll be good to see jekyll getting some love again!

@parkr
Copy link
Member Author

parkr commented Feb 6, 2013

If you have any improvements that you think should be included, maybe you can submit a pull-request against this branch? Then I can check those and merge them into this branch, then merge this branch into our project :)

@markhepburn
Copy link
Contributor

Huh, newbie question -- I was working against the entire mojombo/jekyll, not this extraction. Do you know if there's an easy way to pull my commits + history onto a clone of just jekyll-import? Short of using git-subtree, I suppose? Maybe it's easiest just via a single patch, since it's not exactly a big change :)

@mojombo
Copy link
Contributor

mojombo commented Feb 23, 2013

Merged.

@mojombo mojombo closed this Feb 23, 2013
@jekyll jekyll 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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants