Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Implemented themes_dir configuration option. #1020

Merged
merged 4 commits into from
Feb 14, 2014

Conversation

ThisIsMissEm
Copy link
Contributor

This enables storing themes outside of node_modules and NPM.

Note, ceffd2e means that users are no longer notified if some default themes are out of date, I'm not sure if we really want this or not.

@julianlam julianlam added this to the 0.3.2 milestone Feb 14, 2014
@julianlam julianlam self-assigned this Feb 14, 2014
@julianlam
Copy link
Member

Diff looks fairly straightforward.

Re: ceffd2e -- there is probably a better way to do it (possible related to the recently closed #997)?

julianlam added a commit that referenced this pull request Feb 14, 2014
Implemented `themes_dir` configuration option.
@julianlam julianlam merged commit b6b795d into NodeBB:master Feb 14, 2014
@ThisIsMissEm ThisIsMissEm deleted the themes_dir branch February 14, 2014 17:29
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants