Skip to content

Removes config.theme and restructures how theme variables are cached#3947

Merged
ErisDS merged 1 commit intoTryGhost:masterfrom
hswolff:goodbye-config-theme
Sep 15, 2014
Merged

Removes config.theme and restructures how theme variables are cached#3947
ErisDS merged 1 commit intoTryGhost:masterfrom
hswolff:goodbye-config-theme

Conversation

@hswolff
Copy link
Contributor

@hswolff hswolff commented Sep 3, 2014

resolves #1789

  • removes config/theme.js
  • moves caching of theme variables to api/settings.js which is where the
    rest of the settings cache occurs. this removes the requirement of having
    to push changes to cache, now it simply occurs alongside when settings
    are changed.
  • updates relevant tests.

resolves TryGhost#1789

- removes config/theme.js
- moves caching of theme variables to api/settings.js which is where the
rest of the settings cache occurs.  this removes the requirement of having
to push changes to cache, now it simply occurs alongside when settings
are changed.
- updates relevant tests.
ErisDS added a commit that referenced this pull request Sep 15, 2014
Removes config.theme and restructures how theme variables are cached
@ErisDS ErisDS merged commit b7fdf7d into TryGhost:master Sep 15, 2014
@hswolff hswolff deleted the goodbye-config-theme branch September 16, 2014 01:06
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.

Revisit config.paths.update and config.theme.update

2 participants