pagelet - npm Package Compare versions

Comparing version 0.0.0 to 0.0.1

index.js

@@ -42,4 +42,4 @@ 'use strict';
  *     'methodname',
- *     'methodname'
- *   [,
+ *     'another'
+ *   ],
  *
@@ -49,3 +49,4 @@ *   methodname: function methodname(reply) {
  *   }
- * });
+ * }).on(module);
+ * ```
  *
@@ -79,24 +80,2 @@ * @type {Array}
 /**
- * Optional template engine preference. Useful when we detect the wrong template
- * engine based on the view's file name.
- *
- * @type {String}
- * @public
- */
-Pagelet.writable('engine', '');
-
-/**
- * Get the params of the Page.
- *
- * @type {Object}
- * @public
- */
-Pagelet.readable('params', {
-  enumerable: false,
-  get: function params() {
-    return this.page.params;
-  }
-}, true);
-
-/**
  * Remove the DOM element if we are unauthorized. This will make it easier to
@@ -126,2 +105,25 @@ * create conditional layouts without having to manage the pointless DOM
 /**
+ * The location of your error template. This template will be rendered when:
+ *
+ * 1. We receive an `error` argument from your `render` method.
+ * 2. Your view throws an error when rendering the template.
+ *
+ * If no view has been set it will default to the Pagelet's default error
+ * template which outputs a small HTML fragrment that states the error.
+ *
+ * @type {String}
+ * @public
+ */
+Pagelet.writable('error', '');
+
+/**
+ * Optional template engine preference. Useful when we detect the wrong template
+ * engine based on the view's file name.
+ *
+ * @type {String}
+ * @public
+ */
+Pagelet.writable('engine', '');
+
+/**
  * The location of the Style Sheet for this pagelet. It should contain all the
@@ -210,2 +212,15 @@ * CSS that's needed to render this pagelet. It doesn't have to be a `CSS`
 /**
+ * Get route parameters that we've extracted from the route.
+ *
+ * @type {Object}
+ * @public
+ */
+Pagelet.readable('params', {
+  enumerable: false,
+  get: function params() {
+    return this.page.params;
+  }
+}, true);
+
+/**
  * Reset the instance to it's original state.
@@ -347,2 +362,12 @@ *
 
+  //
+  // Ensure that we have a custom error page for when we fail to render this
+  // fragment.
+  //
+  Pagelet.prototype.error = prototype.error
+    ? path.resolve(dir, prototype.error)
+    : path.resolve(__dirname, 'error.ejs');
+
+  pipe.temper.prefetch(Pagelet.prototype.error, Pagelet.prototype.engine);
+
   if (prototype.css) Pagelet.prototype.css = path.resolve(dir, prototype.css);
@@ -349,0 +374,0 @@ if (prototype.js) Pagelet.prototype.js = path.resolve(dir, prototype.js);

package.json

 {
   "name": "pagelet",
-  "version": "0.0.0",
+  "version": "0.0.1",
   "description": "pagelet",
   "main": "index.js",
   "scripts": {
-    "test": "echo \"Error: no test specified\" && exit 1"
+    "test": "mocha $(find test -name '*.test.js')"
   },
@@ -22,3 +22,3 @@ "repository": {
   },
-  "homepage": "https://github.com/bigpipe/pagelet",
+  "homepage": "http://bigpipe.io",
   "dependencies": {
@@ -29,4 +29,8 @@ "debug": "0.7.x",
   "devDependencies": {
-    "pre-commit": "0.0.x"
+    "chai": "1.8.x",
+    "bigpipe": "git://github.com/bigpipe/bigpipe.git#master",
+    "mocha": "1.13.x",
+    "pre-commit": "0.0.x",
+    "jade": "1.1.x"
   }
 }

Fixed alerts

No README

Quality

Package does not have a README. This may indicate a failed publish or a low quality package.

Found 1 instance in 1 package

No tests

Quality

Package does not have any tests. This is a strong signal of a poorly maintained or low quality package.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

19476

increased by80.27%

Number of package files

8

increased by166.67%

Lines of code

404

increased by16.76%

Number of low quality alerts

1

decreased by-50%

Number of medium quality alerts

1

decreased by-50%

Number of lines in readme file

215

increased byInfinity%

Worsened metrics

Dev dependency count

5

increased by400%

No dependency changes