Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Demo instructions in README are outdated #39

Open
erikcw opened this issue May 27, 2016 · 1 comment
Open

Demo instructions in README are outdated #39

erikcw opened this issue May 27, 2016 · 1 comment

Comments

@erikcw
Copy link

erikcw commented May 27, 2016

A couple of problems with the demo.

  1. The path to the demo from the README is no longer correct. The README instructs new users to cd dist/demo, but the demo directory has been moved to dist/src/demo/demo.js.

  2. When running node demo.js user pw after changing to the above mentioned directory, the following traceback is dumped.

/Users/erik/git/skyweb/dist/src/demo/demo.js:8
var skyweb = new skyweb_1.default();
             ^

TypeError: skyweb_1.default is not a constructor
    at Object.<anonymous> (/Users/erik/git/skyweb/dist/src/demo/demo.js:8:14)
    at Module._compile (module.js:541:32)
    at Object.Module._extensions..js (module.js:550:10)
    at Module.load (module.js:458:32)
    at tryModuleLoad (module.js:417:12)
    at Function.Module._load (module.js:409:3)
    at Function.Module.runMain (module.js:575:10)
    at startup (node.js:160:18)
    at node.js:449:3
@scumdog
Copy link

scumdog commented Aug 22, 2016

Comment out
../skyweb.js line 53 ie "//exports = Skyweb;"

--- a/dist/src/skyweb.js
+++ b/dist/src/skyweb.js
@@ -50,8 +50,8 @@ var Skyweb = (function () {
};
return Skyweb;
}());
-exports = Skyweb;
+//exports = Skyweb;
exports.Skyweb = Skyweb;
Object.defineProperty(exports, "__esModule", { value: true });
exports.default = Skyweb;

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

No branches or pull requests

2 participants