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

Add CSV source #2

Merged
merged 56 commits into from
Dec 30, 2015
Merged

Add CSV source #2

merged 56 commits into from
Dec 30, 2015

Conversation

hvelarde
Copy link
Member

No description provided.

hvelarde and others added 30 commits July 29, 2015 16:57
Import path now uses the same structure as the WordPress site:

- posts and pages are imported according to the permalink_structure
- attachments are imported into the wp-content/uploads folder

Also the following changes were made to the code:

- Update documentation
- Split code in more modules
- Get categories and tags
WordPress allows to include markup on titles and we want to get rid of them.
# separator. To break it into a list, I need to split on
# http and reconstruct the url
# TODO: handle HTTPS scheme
related_urls = ['http{0}'.format(url.rstrip('/'))
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rodfersou you have to fix here

hvelarde added a commit that referenced this pull request Dec 30, 2015
@hvelarde hvelarde merged commit cd0ba41 into master Dec 30, 2015
@hvelarde hvelarde deleted the hvelarde-cvssource branch December 30, 2015 15:40
@gforcada
Copy link
Member

Squashing some commits together would have been nice.

@hvelarde
Copy link
Member Author

yes, but it was getting old and I had no time to review it.

@hvelarde hvelarde mentioned this pull request Jun 16, 2016
@hvelarde hvelarde restored the hvelarde-cvssource branch June 16, 2016 18:50
@hvelarde hvelarde deleted the hvelarde-cvssource branch June 16, 2016 18:59
@hvelarde
Copy link
Member Author

@gforcada reverted and fixed in #4 just to make you happy ;-)

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.

None yet

3 participants