Quick setup on https://github.com/montagejs/montage page issue #1097

Closed
irinamok opened this Issue Jan 21, 2013 · 8 comments

Comments

Projects
None yet
5 participants

Hi guys,

Can you check/change "Quick setup" on https://github.com/montagejs/montage page? It says:

  1. Verify your setup by browsing to http://localhost:8081/montage/test/run.html.

I'm getting

Failed to load resource: the server responded with a status of 404 (Not Found) http://localhost:8081/montage/node_modules/collections/package.json
Should be empty:
Array[26]
q.js:619
Uncaught Can't XHR "http://localhost:8081/montage/node_modules/collections/package.json" q.js:1319

I guess documentation is outdated?

Member

kriskowal commented Jan 21, 2013

Yeah, need to mention that we’re using NPM more now. We should mention that we need to npm install . if you clone Montage instead of using npm install montage. Montage now depends on two external packages.

Contributor

Stuk commented Jan 21, 2013

@francoisfrisch may want to write a guide on using minit now that it does more

The current quick start still seems to be broken, or at least missing something. After following the steps listed, and firing up python -m SimpleHTTPServer, I get a never-ending "Loading" page at http://localhost:8000. Checking the terminal, it seems there are some 404 responses listed:

{16:47}~/git/bonjour ➭ python -m SimpleHTTPServer
Serving HTTP on 0.0.0.0 port 8000 ...
127.0.0.1 - - [11/Jun/2013 16:48:08] "GET / HTTP/1.1" 200 -
127.0.0.1 - - [11/Jun/2013 16:48:09] "GET /assets/style/style.css HTTP/1.1" 200 -
127.0.0.1 - - [11/Jun/2013 16:48:09] code 404, message File not found
127.0.0.1 - - [11/Jun/2013 16:48:09] "GET /favicon.ico HTTP/1.1" 404 -
127.0.0.1 - - [11/Jun/2013 16:48:09] "GET /package.json HTTP/1.1" 200 -
127.0.0.1 - - [11/Jun/2013 16:48:09] "GET /node_modules/montage/package.json HTTP/1.1" 200 -
127.0.0.1 - - [11/Jun/2013 16:48:09] "GET /node_modules/montage/node_modules/collections/package.json HTTP/1.1" 200 -
127.0.0.1 - - [11/Jun/2013 16:48:09] "GET /node_modules/montage/node_modules/mousse/package.json HTTP/1.1" 200 -
127.0.0.1 - - [11/Jun/2013 16:48:09] "GET /node_modules/montage/node_modules/frb/package.json HTTP/1.1" 200 -
127.0.0.1 - - [11/Jun/2013 16:48:09] "GET /ui/welcome.reel/welcome.js HTTP/1.1" 200 -
127.0.0.1 - - [11/Jun/2013 16:48:09] "GET /ui/welcome.reel/welcome.html HTTP/1.1" 200 -
127.0.0.1 - - [11/Jun/2013 16:48:09] "GET /assets/images/montage-logo-gradients.svg HTTP/1.1" 200 -
127.0.0.1 - - [11/Jun/2013 16:48:09] code 404, message File not found
127.0.0.1 - - [11/Jun/2013 16:48:09] "GET /node_modules/montage/ui/dynamic-text.reel/dynamic-text.js HTTP/1.1" 404 -

I also performed (since it was mentioned to do so not in the documentation, but after minit create:app --name hello) a npm install . command in the project folder. The same results followed.

Member

kriskowal commented Jun 11, 2013

@jdhorner If you could show us npm ls, that might provide some insight.

I'm currently using Node.js v0.10.10

{16:59}~/git/hello ➭ ls  
assets  index.html  node_modules  package.json  README.md  run-tests.html  testacular.conf.js  ui
{16:59}~/git/hello ➭ npm ls
npm WARN package.json hello@0.1.0 No repository field.
npm WARN package.json montage-testing@0.1.3 No repository field.
npm WARN package.json montage@0.12.7 No repository field.
npm WARN package.json jsdom@0.2.19 'repositories' (plural) Not supported.
npm WARN package.json Please pick one as the 'repository' field
npm WARN package.json assert-plus@0.1.2 No repository field.
npm WARN package.json ctype@0.5.2 No repository field.
hello@0.1.0 /home/jdhorner/git/hello
├─┬ montage@0.13.0
│ ├── collections@0.1.20
│ ├── frb@0.2.3
│ ├─┬ htmlparser2@3.0.5
│ │ ├── domelementtype@1.1.1
│ │ ├── domhandler@2.0.3
│ │ ├── domutils@1.1.2
│ │ └── readable-stream@1.0.2
│ └─┬ mousse@0.1.3
│   └── q@0.8.12
└─┬ montage-testing@0.1.3
  └─┬ montage@0.12.7
    ├── collections@0.1.20
    └─┬ jsdom@0.2.19
      ├─┬ contextify@0.1.5
      │ └── bindings@1.1.0
      ├── cssom@0.2.5
      ├── cssstyle@0.2.3
      ├── htmlparser@1.7.6
      └─┬ request@2.21.0
        ├── aws-sign@0.3.0
        ├── cookie-jar@0.3.0
        ├── forever-agent@0.5.0
        ├─┬ form-data@0.0.8
        │ ├── async@0.2.9
        │ └─┬ combined-stream@0.0.4
        │   └── delayed-stream@0.0.5
        ├─┬ hawk@0.13.1
        │ ├─┬ boom@0.4.2
        │ │ └── hoek@0.9.1
        │ ├── cryptiles@0.2.1
        │ ├── hoek@0.8.5
        │ └─┬ sntp@0.2.4
        │   └── hoek@0.9.1
        ├─┬ http-signature@0.9.11
        │ ├── asn1@0.1.11
        │ ├── assert-plus@0.1.2
        │ └── ctype@0.5.2
        ├── json-stringify-safe@4.0.0
        ├── mime@1.2.9
        ├── node-uuid@1.4.0
        ├── oauth-sign@0.3.0
        ├── qs@0.6.5
        └── tunnel-agent@0.3.0

It does appear that it's a case of dynamic-text.reel missing from montage/ui within the montage node-module:

{17:06}~/git/hello/node_modules/montage/ui ➭ ls -Falh
total 156K
drwxr-xr-x 11 jdhorner jdhorner 4.0K Jun 11 16:58 .
drwxr-xr-x  9 jdhorner jdhorner 4.0K Jun 11 16:58 ..
drwxr-xr-x  2 jdhorner jdhorner 4.0K Jun 11 16:58 base
-rw-r--r--  1 jdhorner jdhorner 105K Jun 11 01:59 component.js
-rw-r--r--  1 jdhorner jdhorner 2.2K Jun 11 01:59 component.meta
drwxr-xr-x  2 jdhorner jdhorner 4.0K Jun 11 16:58 condition.reel
drwxr-xr-x  2 jdhorner jdhorner 4.0K Jun 11 16:58 flow.reel
drwxr-xr-x  2 jdhorner jdhorner 4.0K Jun 11 16:58 loader.reel
drwxr-xr-x  2 jdhorner jdhorner 4.0K Jun 11 16:58 overlay.reel
drwxr-xr-x  2 jdhorner jdhorner 4.0K Jun 11 16:58 repetition.reel
drwxr-xr-x  2 jdhorner jdhorner 4.0K Jun 11 16:58 slot.reel
drwxr-xr-x  2 jdhorner jdhorner 4.0K Jun 11 16:58 substitution.reel
drwxr-xr-x  2 jdhorner jdhorner 4.0K Jun 11 16:58 text.reel
Collaborator

aadsm commented Jun 11, 2013

@jdhorner we just released a new version of montage (v0.13) and now dynamic-text is just called text.

In a sense this is a big release because we changed some component names and moved others around to have a more consistent API. We're still working on updating the examples and even add more to reflect these new changes.
If you find more problems please report them as we're still making sure that everything works correctly.

You can read an overview of the changes here: https://github.com/montagejs/montage/blob/master/CHANGES.md

@aadsm Makes sense! Thanks for the info.

aadsm closed this Jun 12, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment