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

Does not work at all for me #42

Closed
guidoffm opened this issue Oct 19, 2015 · 9 comments
Closed

Does not work at all for me #42

guidoffm opened this issue Oct 19, 2015 · 9 comments

Comments

@guidoffm
Copy link

1st problem: After installing bootprint and bootprint swagger when I start bootprint I get a message complaining about missing "handelbars". There might be a missing dependency to handlebars. After installing handlebars I these versions:

├─┬ [email protected]
│ └─┬ [email protected]
│ └── [email protected]
├── [email protected]
├─┬ [email protected]
│ └── [email protected]
└── [email protected]
After calling
bootprint swagger http://localhost:53108/backend/swagger/docs/v1 neu
I get:

Loading bootprint-swagger 0.11.0
Loading bootprint-json-schema 0.8.1
Loading bootprint-base 0.6.1
[object Object]

No output is generated at all.

@nknapp
Copy link
Member

nknapp commented Oct 19, 2015

Hi @guidoffm, thanks for taking the time to report this.
I have unpublished [email protected]. The latest changes seem to be broken. I've tested this with the petstore-example and it seems to work. I'll investigate further tonight.
Mind trying again?

@guidoffm
Copy link
Author

Hi Nils,

the issue with missing handlebars dependency is gone. npm ls -g now reads:
├─┬ [email protected]
│ └─┬ [email protected]
│ └── [email protected]
├─┬ [email protected]
│ └─┬ [email protected]
│ └─┬ [email protected]
│ └── [email protected]
└─┬ [email protected]
└── [email protected]

After calling
bootprint swagger http://localhost:53108/backend/swagger/docs/v1 neu
I get:

Loading bootprint-swagger 0.11.0
Loading bootprint-json-schema 0.8.1
Loading bootprint-base 0.6.0
[object Object]

Still no output generated at all.

@nknapp
Copy link
Member

nknapp commented Oct 19, 2015

Can you try the petstore-example: bootprint swagger http://petstore.swagger.io/v2/swagger.json target?

@nknapp
Copy link
Member

nknapp commented Oct 19, 2015

@guidoffm I have published the modules again tonight, I tried

npm -g install bootprint
npm -g install bootprint-swagger

bootprint --version # prints "0.7.7"
bootprint swagger http://petstore.swagger.io/v2/swagger.json target

and I get the following result.

Loading bootprint-swagger 0.11.1
Loading bootprint-json-schema 0.8.3
Loading bootprint-base 0.6.2
[ 'target/index.html', 'target/main.css', 'target/main.css.map' ]

The target/index.html-file looks good. I would like to verify this with your setup, because I don't understand why your bootprint-call prints [object Object]. But since I cannot reproduce your situation (because I don't have access to your swagger.json), I would like you to reproduce mine.

Could you try the new modules with your own swagger.json, but also with the petstore-example?

@jneubeck
Copy link

After uninstalling everything and reinstalling I get the following error

bootprint swagger http://petstore.swagger.io/v2/swagger.json target
Loading bootprint-swagger 0.11.1
Loading bootprint-json-schema 0.8.3
Loading bootprint-base 0.6.2

C:\Users\XXXX\AppData\Roaming\npm\node_modules\bootprint\node_modules\q\q.js:155
                throw e;
                ^
Error: The partial base/title could not be found
    at Object.invokePartial (C:\Users\XXXXX\AppData\Roaming\npm\node_modules\bootprint\node_modules\customize-engine-handlebars\node_modules\handlebars\dist\cjs\handlebars\runtime.js:216:11)
    at Object.invokePartialWrapper [as invokePartial] (C:\Users\XXXXX\AppData\Roaming\npm\node_modules\bootprint\node_modules\customize-engine-handlebars\node_modules\handlebars\dist\cjs\handlebars\runtime.js:61:39)
    at Object.eval (eval at createFunctionContext (C:\Users\XXXXX\AppData\Roaming\npm\node_modules\bootprint\node_modules\customize-engine-handlebars\node_modules\handlebars\dist\cjs\handlebars\compiler\javascript-compiler.js:221:23), <anonymous>:6:23)....

@nknapp
Copy link
Member

nknapp commented Oct 21, 2015

@jneubeck Firstly I have to apologize for the long installing time of bootprint-swagger. npm@1 and npm@2 download all peer-dependencies, so the bootprint package has been downloaded three times. I have removed the dependency for now.
The bug you observe is a regression that was introduced in [email protected]. The partials are not registered correctly because Windows separates path-components with a backslash and not a slash.
I will fix that in a moment.

nknapp pushed a commit to bootprint/customize that referenced this issue Oct 21, 2015
…returns paths in POSIX-style,

that are separated by slashes and not backslashed (even on Windows)
(see [bootprint-swagger#42](bootprint/bootprint-openapi#42 (comment)))
@nknapp
Copy link
Member

nknapp commented Oct 21, 2015

@jneubeck I have released a new version of customize which is responsible for building the file-paths.
Would you mind trying again? The just make sure that npm ls -g show [email protected], otherwise you won't have the fix installed.

@jneubeck
Copy link

Thank you it now works.

@nknapp
Copy link
Member

nknapp commented Oct 21, 2015

Glad to hear

@nknapp nknapp closed this as completed Oct 22, 2015
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

3 participants