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

Issue 13 read zip #1

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Issue 13 read zip #1

wants to merge 4 commits into from

Conversation

ErisDS
Copy link
Owner

@ErisDS ErisDS commented Aug 23, 2016

testing

ErisDS added 4 commits August 23, 2016 18:05
closes TryGhost#13

- use `index.hbs` to detect where in the zip file the theme lives
- add some tests to show how this works
- call fetch on boot, it will likely be done by the time the first request comes in
- if it isn't, it doesn't really matter
- then call fetch again when the ttl runs out
- TODO: do this some better way
- add express-hbs & handlebars helpers that were missed
- add old and brand-new Ghost helpers so that we support the latest version
- Refactor how the zip is deleted, to try to prevent the app from crashing
- Add error handling
- Add forced default layout
- Improve result output
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.

1 participant