-
Notifications
You must be signed in to change notification settings - Fork 9
report() bug reproduced code. lifecycle, viewChild and angular syntax. #13
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
Conversation
/** | ||
* Error due to `@ionic/angular/standalone` not containing a lifecycle. | ||
*/ | ||
export class LifecyclePage implements OnInit, ViewWillEnter, ViewDidEnter, ViewWillLeave, ViewDidLeave { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Error due to @ionic/angular/standalone
not containing a lifecycle.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks to be a missing set of exports from Ionic Framework. This will be addressed here: ionic-team/ionic-framework#28346!
apps/angular/ionic-angular-standalone/src/app/issue/view-child/view-child.page.ts
Show resolved
Hide resolved
apps/angular/ionic-angular-standalone/src/app/issue/not-import-one/not-import-one.page.html
Show resolved
Hide resolved
Thank you so much for the reproduction cases and testing! I appreciate you! |
I'm going to close this PR out now that all the feedback has been addressed 👍 the lifecycle interfaces will be available when we release v7.5.1 of Let me know if you run into any other issues or have ideas for improvement. I'll put some thought to a better reporter for manual migration steps. |
No merging is required. Reproducible code submission.