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 support for importing images #4690

Merged
merged 1 commit into from
Dec 24, 2014
Merged

Conversation

ErisDS
Copy link
Member

@ErisDS ErisDS commented Dec 20, 2014

closes #4608, #4609

  • image handler loads in any image files & figures out where they'll get stored
  • image importer has a preprocessor which replaces image paths in
    pertinent spots of post, tag and user models
  • image importer stores images, keeping the path where it makes sense
  • basic test for the preprocessor

@ErisDS ErisDS force-pushed the image-importer branch 2 times, most recently from ce680e1 to 2ad7eca Compare December 20, 2014 23:16
@ErisDS
Copy link
Member Author

ErisDS commented Dec 20, 2014

Awkward moment there where it turns out one of the previous tests is overriding the config and not resetting it, but got it fixed, should pass this time.

closes TryGhost#4608, TryGhost#4609

- image handler loads in any image files & figures out where they'll get stored
- image importer has a preprocessor which replaces image paths in
  pertinent spots of post, tag and user models
- image importer stores images, keeping the path where it makes sense
- basic test for the preprocessor
jaswilli added a commit that referenced this pull request Dec 24, 2014
Add support for importing images
@jaswilli jaswilli merged commit 2c66430 into TryGhost:master Dec 24, 2014
ErisDS added a commit to ErisDS/Ghost that referenced this pull request Jan 2, 2015
ref TryGhost#4608, TryGhost#4609, TryGhost#4690

- fix errors with cleaning up files
- improve handling of base directories, and introduce a simple valid format for zips (must contain importable files or folders, and may contain up to one base directory)
- vastly improve test coverage
@ErisDS ErisDS deleted the image-importer branch January 12, 2015 14:19
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.

Ghost Importer: Find and copy any image files
2 participants