Permalink
Browse files

the instantiate: false property is no longer required when using impr…

…ove, also depend on a newer moog-require that has proper behavior for asset chains and autoloading with improve
  • Loading branch information...
1 parent ae0e3b6 commit bc8389e5885d16bb1c8b82d7f244f8304b265a33 @boutell boutell committed Dec 15, 2016
Showing with 8 additions and 3 deletions.
  1. +4 −0 CHANGELOG.md
  2. +2 −1 index.js
  3. +2 −2 package.json
View
@@ -1,5 +1,9 @@
# Changelog
+## 2.11.1
+
+Developers are no longer required to explicitly set `instantiate: false` when adding an npm module that uses the `improve` property to enhance a different module. In addition, bugs were fixed in the underlying `moog-require` module to ensure that assets can be loaded from the `public` and `views` folders of modules that use `improve`.
+
## 2.11.0
All tests passing.
View
@@ -247,7 +247,8 @@ module.exports = function(options) {
function instantiateModules(callback) {
self.modules = {};
return async.eachSeries(_.keys(self.options.modules), function(item, callback) {
- if (self.options.modules[item] && (self.options.modules[item].instantiate === false)) {
+ var improvement = self.synth.isImprovement(item);
+ if (self.options.modules[item] && (improvement || self.options.modules[item].instantiate === false)) {
// We don't want an actual instance of this module, we are using it
// as an abstract base class in this particular project (but still
// configuring it, to easily carry those options to subclasses, which
View
@@ -1,6 +1,6 @@
{
"name": "apostrophe",
- "version": "2.11.0",
+ "version": "2.11.1",
"description": "The Apostrophe Content Management System.",
"main": "index.js",
"scripts": {
@@ -48,7 +48,7 @@
"mkdirp": "^0.5.0",
"moment": "^2.9.0",
"mongodb": "^2.0.0",
- "moog-require": "^0.3.0",
+ "moog-require": "^0.4.0",
"nunjucks": "^2.0.0",
"oembetter": "^0.1.10",
"passport": "^0.3.0",

0 comments on commit bc8389e

Please sign in to comment.