-
Notifications
You must be signed in to change notification settings - Fork 13.5k
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
RC.1 angular build fails in production mode #17034
Comments
We must add assets at angular.json in Ionic 4 RC.1. |
@rdlabo Yes, this is mentioned in the changelog ( https://github.com/ionic-team/ionic/blob/master/CHANGELOG.md#breaking-changes ). But if you run in production mode, icons not working ;) |
@paulstelzer Oh. Icon worked in production build in my environment... Maybe cache worked..? |
did you upload it to your server and tried again (cache cleared)? |
@paulstelzer icons work for me, but I have the error for slides: |
@manucorporat is already working on it and will add some more tests so something like that will hopefully not happen again :) |
We hope to solve the problem as soon as possible. The product must be delivered and launched in these two days, because this problem cannot be used urgently |
The problem is already solved. If you need it right now, just fork this repo, install all packages and build it. Then you just need to copy it into your node modules. |
@paulstelzer I have build @ionic/angular and copied the package into my node_modules. The issue is still there. Im doing something wrong? |
could you explain which issue? The ion-slides issue still exist (see #17085 ). But everything else should work, so only thing you can't do is using the slides Or you revert back to rc.0 if you need a build |
Yep, the ion-slide issue. |
@orlyapps the fix for slides is now integrated in the master branch |
Everyone who wants to try it out, youcan install the latest dev build by |
Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of Ionic, please create a new issue and ensure the template is fully filled out. |
Bug Report
This issue is fixed already and rc.2 will coming soon!
Ionic version:
[x] 4.0.0-rc.1
Current behavior:
rc.1 isn't working in production mode (
ng build --prod
/ionic build --prod
). if you run without --prod , everything is workingThere are already some issues about (see for there console logs):
Unhandled Promise rejection: c.data is not a function ; Zone: <root> ; Task: Promise.then ; Value: TypeError: c.data is not a function
(#17032 / #17033 )#17029
Other errors are:
I will close them so we just have one issue about this topic. It's not only related to
@ViewChild()
, ion button disabled is also not working and slides only shows the first sideI did some resarch and the issue must be related in #16966. If I build with this PR, production isn't working anymore. I can be wrong, so everyone can do tests too :)
Expected behavior:
Should work in production build too
Steps to reproduce:
Add e.g. @ViewChild and try to access to it. Or add a split pane and a menu.
The text was updated successfully, but these errors were encountered: