-
Notifications
You must be signed in to change notification settings - Fork 52
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
ionic app uses old capacitor plugin style (fails) #272
Comments
@wstidolph can you confirm that your project is using 12.4.3 of xplat latest? Ionic latest was added in 12.4.1 specifically: If you update to 12.4.3, or run |
Yeah, I was on 12.4.0. Just to verify: with 12.4.3 and generating a default ionic app (which runs!) I see that the recommended @capacitor packages are installed, but the generated ionic app doesn't use Capacitor at all, does it? So it's up to me to use it if I choose? |
@wstidolph it should by default use Capacitor and should have @capacitor/... deps in it. When you can you might drop the Ionic app's package.json here that you generated in your workspace and I can help confirm if it's correct. |
(EDIT: replace email reply with same content, just so Markdown supported) The
The previous generated code had the StatusBar used, so I re-added that this
|
Ah got it now thanks. Yes we can add that back to the initial setup of each app. The general plan is to swap the ionic app generator and use https://nxtend.dev/docs/ionic-angular/getting-started/ under the hood as external schematic to streamline the Ionic handling a bit more. |
(xplat 12.4.0)
The generated Ionic app Capacitor starts but doesn't work because the imports and status bar styling in
app.component.ts
need to be brought up to the new Capacitor 3 method:and the
StatusBar.setStyle
call ininitializeApp
should become:This required I install of @capacitor/status-bar - when fixing that you might want to just add in some other packages: per https://capacitorjs.com/docs/updating/3-0:
The text was updated successfully, but these errors were encountered: