-
Notifications
You must be signed in to change notification settings - Fork 311
Template paths are not resolved correctly #160
Comments
I have the same problem ( |
me too |
I've found a quickfix, see #162. You can easily apply it locally in your node_modules for now until the Ionic team brings us an updated version :). |
For me, it actually fails with with a correct path: Any ideas? It worked just fine before the update. Edit: I've applied the fix above, but it's having no effect, the path it's complaining about is still the same. |
@danielcrk-cn You see no other errors? I saw the error you mentioned as well, but after fixing the errors before it, this last error also disappeared. |
@Manduro I do, my project is old and converted so it has many typescript errors (that are not breaking errors. though), so the only thing I did was to circumvent the The last lines look like this:
|
@danielcrk-cn Weird! Could be a separate issue then. |
@Manduro Ok, i've just tried this: Create a new ionic started project and in /pages/home/home.ts, create a simple ts-error, like adding Now, in transpile.ts, make the change to ignore the errors, e.g. like this:
To me, this looks like it should resolve and proceed despite the errors, but it doesn't. Bundle process starts but never finishes. Could you pleeease help me getting this to work? Thanks! Edit: Before todays update, it was building fine so worst case it, I have to go back to the previous version. |
@danielcrk-cn My advice would be to fix those TS errors instead of ignoring them. If you can't fix them you can ask for help on the Ionic forums. |
@Manduro I know, but sometimes, reality and best practice do not match. It's a huge project that has tons of errors like the one above, and I just need to get the application running, and hopefully have the time later on to fix the ts-errors. What would be the easiest way to just ignore the errors for now? Can you see a problem width the way i've tried to do it? |
@danielcrk-cn I don't know how to fix your problem, I'm sorry. I do know that this is not the place for your questions, as it's unrelated to this issue. You should take this to the forums to ask for help, or create a feature request in a new issue. |
@Manduro and co, Can you open
Note the commented out line. I don't think this line is needed. Can you do that and test and let me know? Thanks, |
@danbucholtz it worked |
Thanks! Can a few others test to validate quick and let me know? I want to push this out today. Thanks, |
I can confirm it's working after the fix. 👍 |
Getting this https://forum.ionicframework.com/t/ionic-serve-broken-after-upgrading-to-rc-1/66976/21?u=itlr after apply the hack. My tsconfig.json
|
@danbucholtz I commented the line Is there any way to execute ionic run browser in dev mode instead prod? I am running cordova run browser. Thanks |
Are you on windows by chance? I haven't been able to recreate this so this fix is largely guessing. We are about to publish a new version so once it's out we'll have the masses test it out and see how it goes. Thanks, |
how do you make a release before testing all features! ionic serve is broken because of this issue right now. |
@aemr3, we test like crazy. Unless users can provide clear reproductions of issues, they're very hard to track and down and fix. The fix I am referencing above have had 4 other developers on Github tell us this change fixes the issue for them. Thanks, |
@danbucholtz Yes, windows. In prod mode works fine, the issue is just happening on dev |
@danbucholtz I have 404 errors to main.js twice and one on main.css with or without comment the line 49 in both modes, dev and prod. Cordova CLI: 6.3.1
|
Commenting out line 49 in transpile.js did not solve the problem for me. Same "src/src" problem.
|
We released a new CLI and a new app-scripts
I'm going to close this issue. Please let me know if this fixes it. I'll re-open if needed. Thanks, |
@danbucholtz I believe an underlying issue has not yet been resolved. All ts files in the project folder are being transpiled instead of just the ones inside the src folder. I think this is also why you couldn't reproduce this: you didn't have any ts files outside your src folder. When I log the filenames of the ts files that are being transpiled I see the following:
These config files are copied to the src folder on watch/build, based on the environment I'm running, so they should not be transpiled by app-scripts. |
I updated the ionic-app-scripts latest version and i have the 'src' duplicated issue again. Commented it out I get main.js 404 twice and main.scss once. |
What is the Thanks, |
I think that's a good idea. I will look into that this morning. How are you compiling the Thanks, |
@danbucholtz My other .ts files are never compiled while they are outside of the In my opinion everything that should be compiled should be in the |
@danbucholtz finally it is working. Update failed, once i fix the update to 0.0.36 the issue was fixed. Thanks |
Thanks, we'll probably take that approach, then. Thanks, |
We updated the https://github.com/driftyco/ionic2-app-base/blob/master/tsconfig.json#L15 |
Done. Now I get this:
|
Can you paste the rest of your console output? It seems like the Thanks, |
I don't see any .js files in /.tmp now. It's just a copy of the .html|.ts files. Here's the full output:
|
Any chance you can share your project with us to test this out? Thanks, |
It's in a private repo, and requires a connection to a non-public API. For building purposes you wouldn't need the connection to the API I guess. I'm open to a screen share if you want to right now. I'll ping you on Slack. |
@danbucholtz Nice, works for me! Maybe the |
Hey there,
I created the project using The npm-debug.log contains the following:
EDIT: Seems to be caused by the hello-world/tabs project. When I use an older Ionic2 project everything works fine. |
@XPanniX update your node version. I suggest you to use nvm to maanage nodejs. |
@marcoturi Seems like I had node installed in several locations and different versions. Removing all but the latest nvm one did the trick. Thank you very much! |
Short description of the problem:
The watch task fails to start with a lot of errors. It seems to be using a wrong path (
src/src
). It worked on 0.0.30, but fails on 0.0.31 and up.It also seems unable to find
main.dev.ts
, while the path that is logged is correct.Which @ionic/app-scripts version are you using?
0.0.33
The text was updated successfully, but these errors were encountered: