Skip to content

fix(cli-plugin-ionic1): use documentRoot and watchPatterns from ionic…#2314

Closed
norbertesperto wants to merge 1 commit intoionic-team:masterfrom
twogather:master
Closed

fix(cli-plugin-ionic1): use documentRoot and watchPatterns from ionic…#2314
norbertesperto wants to merge 1 commit intoionic-team:masterfrom
twogather:master

Conversation

@norbertesperto
Copy link
Copy Markdown

use documentRoot and watchPatterns from ionic.config.json

In ionic1 projects the documentRoot and watchPatterns config params are no longer working. This pull request aims to fix this.

@imhoffd
Copy link
Copy Markdown
Contributor

imhoffd commented May 18, 2017

@norbert2gather This is looking good. documentRoot is not used anywhere else in the CLI, either. You may have noticed several TODOs in the code reminding me to fix the hard-coded www directory.

I will test it soon. Thanks for submitting the PR.

@imhoffd imhoffd added this to the CLI Next milestone May 18, 2017
@imhoffd
Copy link
Copy Markdown
Contributor

imhoffd commented May 19, 2017

Github didn't pick up the merge: a878a95

Thanks @norbert2gather !!! 🎉

@imhoffd imhoffd closed this May 19, 2017
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.

2 participants