New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MovableType export: adding comments #66

Closed
wants to merge 3 commits into
from

Conversation

Projects
None yet
4 participants
@shigeya
Contributor

shigeya commented Nov 2, 2013

I have created a set of patches which allow me to export comments from MovableType site.

The code here adds some IDs to files generated in _posts directory, and also exports comments in _comments directory.

By applying templates and plugin (https://github.com/shigeya/jekyll-static-comments/tree/mt_static_comments) to the output of this modified plugin, I can statically generate comments to each posts, from the export origin site.

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Dec 11, 2013

Member

This is pretty cool.

Member

parkr commented Dec 11, 2013

This is pretty cool.

@shigeya

This comment has been minimized.

Show comment
Hide comment
@shigeya

shigeya Dec 12, 2013

Contributor

I've finally start using sites creating this plugin and template and pretty much satisfied. I think it is worth to merge.

Contributor

shigeya commented Dec 12, 2013

I've finally start using sites creating this plugin and template and pretty much satisfied. I think it is worth to merge.

@mattr-

This comment has been minimized.

Show comment
Hide comment
@mattr-

mattr- Dec 12, 2013

Member

👍 from me.

Member

mattr- commented Dec 12, 2013

👍 from me.

@mattr-

This comment has been minimized.

Show comment
Hide comment
@mattr-

mattr- Dec 12, 2013

Member

@shigeya Would you mind rebasing this to current master and force pushing to your branch so we can do an automatic merge?

Member

mattr- commented Dec 12, 2013

@shigeya Would you mind rebasing this to current master and force pushing to your branch so we can do an automatic merge?

@shigeya

This comment has been minimized.

Show comment
Hide comment
@shigeya

shigeya Dec 12, 2013

Contributor

Sure. Work on it right now.

Contributor

shigeya commented Dec 12, 2013

Sure. Work on it right now.

@shigeya

This comment has been minimized.

Show comment
Hide comment
@shigeya

shigeya Dec 12, 2013

Contributor

Hmm. you've changed overall structure a lot. will require little more time than I expected. Will work on this in a day or two and update this pull request. (Merge done but can't verify the result since invocation scheme is now different..)

Contributor

shigeya commented Dec 12, 2013

Hmm. you've changed overall structure a lot. will require little more time than I expected. Will work on this in a day or two and update this pull request. (Merge done but can't verify the result since invocation scheme is now different..)

@shigeya

This comment has been minimized.

Show comment
Hide comment
@shigeya

shigeya Dec 22, 2013

Contributor

Don't have enough to work on this.. After I pushed I realised I need to change option parsing. will push another one soon.

Contributor

shigeya commented Dec 22, 2013

Don't have enough to work on this.. After I pushed I realised I need to change option parsing. will push another one soon.

Comment file generation for MovableType
- comments option added (currently default false)
- metadata for entry added: blog_id, post_id, mt_id, basename
  note: post_id is same as mt_id. for the comment plugin
- comments formats are same as posts: with YAML front matter
- comment related metadata/content output methods and path generators
- also fixes default option processing order
@shigeya

This comment has been minimized.

Show comment
Hide comment
@shigeya

shigeya Dec 22, 2013

Owner

This line should be before all options.fetch calls. I added fixes to this, too.

This line should be before all options.fetch calls. I added fixes to this, too.

@shigeya

This comment has been minimized.

Show comment
Hide comment
@shigeya

shigeya Dec 22, 2013

Contributor

Fixed all (including comments). Produces exactly same output as my previous version does.

Contributor

shigeya commented Dec 22, 2013

Fixed all (including comments). Produces exactly same output as my previous version does.

Show outdated Hide outdated lib/jekyll-import/importers/mt.rb
@@ -21,7 +21,8 @@ def self.default_options
:blog_id => nil,
:categories => true,
:dest_encoding => 'utf-8',
:src_encoding => 'utf-8'
:src_encoding => 'utf-8',
:comments => false

This comment has been minimized.

@parkr

parkr Dec 22, 2013

Member

These default options are going to have to use string keys or the values won't override!

@parkr

parkr Dec 22, 2013

Member

These default options are going to have to use string keys or the values won't override!

This comment has been minimized.

@shigeya

shigeya Dec 22, 2013

Contributor

Oh, then, this code was not working anyway now... Will change all of then and will push..

@shigeya

shigeya Dec 22, 2013

Contributor

Oh, then, this code was not working anyway now... Will change all of then and will push..

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Dec 22, 2013

Member

Great! Just one comment about then it's good to 🚢.

Member

parkr commented Dec 22, 2013

Great! Just one comment about then it's good to 🚢.

@shigeya

This comment has been minimized.

Show comment
Hide comment
@shigeya

shigeya Dec 22, 2013

Contributor

Replaced option related symbols to string

Contributor

shigeya commented Dec 22, 2013

Replaced option related symbols to string

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Dec 22, 2013

Member

Great! This LGTM. Would you mind quickly adding the new comments option to the movable type docs page? It's site/docs/mt.md, I think.

Member

parkr commented Dec 22, 2013

Great! This LGTM. Would you mind quickly adding the new comments option to the movable type docs page? It's site/docs/mt.md, I think.

@shigeya

This comment has been minimized.

Show comment
Hide comment
@shigeya

shigeya Dec 23, 2013

Contributor

Hi, added docs. Happy holidays:)

Contributor

shigeya commented Dec 23, 2013

Hi, added docs. Happy holidays:)

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Dec 23, 2013

Member

LGTM!! @mattr-?

Happy holidays :)

Member

parkr commented Dec 23, 2013

LGTM!! @mattr-?

Happy holidays :)

@parkr parkr closed this in 881640a Dec 26, 2013

parkr added a commit that referenced this pull request Dec 26, 2013

@shigeya

This comment has been minimized.

Show comment
Hide comment
@shigeya

shigeya Dec 26, 2013

Contributor

Thank you:)

Contributor

shigeya commented Dec 26, 2013

Thank you:)

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Dec 26, 2013

Member

Thank YOU! Great work on this PR :)

Member

parkr commented Dec 26, 2013

Thank YOU! Great work on this PR :)

parkr added a commit that referenced this pull request Jan 11, 2014

parkr added a commit that referenced this pull request Apr 14, 2016

@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017

antonizoon pushed a commit to antonizoon/jekyll-import that referenced this pull request Jan 22, 2018

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