Skip to content
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

Wordpress XML importer now handle drafts #346

Closed
wants to merge 1 commit into from
Closed

Wordpress XML importer now handle drafts #346

wants to merge 1 commit into from

Conversation

sametmax
Copy link
Contributor

Now pelican_import.py also import draft from wordpress XML export and add the proper metadata to it.

I created the required unit tests to back this up, but since I'm an newcomer in this project, a peer review would be appreciated.

This patch also solves the same issue that the patch (#345) does, but not in the same way (importing accept empty posts if they are drafts), so I made each of these pull requests independant in case you want to keep one and not the other.

@sametmax
Copy link
Contributor Author

For tests.support.mute (https://github.com/sametmax/pelican/commit/e783acd51453bdee5c3a76148ebc36fa99798e7b#L2R72), I forgot to mention stdout hijacking happens in all the threads in its docstring. You might want to add it to the docstring to avoid somebody misusing it.

@almet
Copy link
Member

almet commented Oct 25, 2012

Closing as the remote doesn't exist anymore.

@almet almet closed this Oct 25, 2012
@arthurlutz
Copy link

This feature would be appreciated.

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