Skip to content

Conversation

jimthedev
Copy link

When running gulp demos, the demos look for files like /dist/ionic-demo/nightly/ionic/css/ionic.min.css
The current instructions direct devs to start their python server at /dist/ionic-demo/.
This commit instructs devs to create a symlink to the parent directory so that the current build of ionic can be used in the demos.

When running gulp demos, the demos look for files like /dist/ionic-demo/nightly/ionic/css/ionic.min.css
The current instructions direct devs to start their python server at /dist/ionic-demo/.
This commit instructs devs to create a symlink to the parent directory so that the current build of ionic can be used in the demos.
@perrygovier
Copy link
Collaborator

Thanks for bringing this up @jimthedev. While this works, I'm inclined to update the demos so that the python server can be run from the root ionic folder like tests can. I'm not sure if this would break any of our automation though. @ajoslin, if it's safe to update the src paths in the demos, please assign this back.

@jimthedev
Copy link
Author

Yeah I agree that would be ideal. Didn't know about other tooling conflicts. If moving the test locations would be better then I will close this PR. Will leave open for now.

@perrygovier
Copy link
Collaborator

Hey @jimthedev, I went ahead and updated the gulp task to generate the right links in the first place. Should be good now. Thanks for bringing it to our attention.
7d000de

@jimthedev
Copy link
Author

Great! Thank you!

On Mon, Sep 22, 2014 at 6:29 PM, Perry Govier notifications@github.com
wrote:

Closed #2238.

Reply to this email directly or view it on GitHub:
#2238 (comment)

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

Successfully merging this pull request may close these issues.

3 participants