[exs] Updated `index.js` for cloud deployment, the dependencies in `package.json`, module names for mojits. #1353

Merged
merged 16 commits into from Feb 28, 2014

Projects

None yet

2 participants

@zhouyaoji

No description provided.

@imalberto imalberto commented on an outdated diff Feb 27, 2014
examples/sandbox/view_refresh/app.js
+
+
+/*jslint node:true*/
+
+'use strict';
+
+var debug = require('debug')('app'),
+ express = require('express'),
+ mojito = require('../../../'),
+ app;
+
+app = express();
+
+app.use(mojito.middleware());
+app.mojito.attachRoutes();
+app.post('/tunnel', mojito.tunnelMiddleware());
@imalberto
imalberto Feb 27, 2014

This explicit tunnel call is no longer required. That's taken care of by mojito.middleware().

@imalberto imalberto commented on an outdated diff Feb 27, 2014
examples/sandbox/metabubble/application.json
@@ -1,6 +1,7 @@
[
{
"settings": ["master"],
+ "appPort": 8666,
@imalberto
imalberto Feb 27, 2014

appPort no longer valid in application.json.

@imalberto imalberto commented on an outdated diff Feb 27, 2014
examples/sandbox/metabubble/app.js
+
+
+/*jslint node:true*/
+
+'use strict';
+
+var debug = require('debug')('app'),
+ express = require('express'),
+ mojito = require('../../../'),
+ app;
+
+app = express();
+mojito.extend(app);
+app.use(mojito.middleware());
+app.mojito.attachRoutes();
+app.post('/tunnel', mojito.tunnelMiddleware());
@imalberto
Yahoo Inc. member

@zhouyaoji Looking good. Just remove references to appPort in application.json and mijito.tunnelMiddleware() in app.js since those are no longer supported.

@imalberto
Yahoo Inc. member

+1. Merge it :)

@zhouyaoji zhouyaoji merged commit 97a78e8 into yahoo:develop Feb 28, 2014

1 check passed

Details default The Travis CI build passed
@zhouyaoji zhouyaoji deleted the zhouyaoji:next_deploy_to_cloud branch Feb 28, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment