Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Answer:52 lazy load component #964

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

LMFinney
Copy link
Contributor

Checklist for challenge submission

  • Start your PR message with Answer:${challenge_number}

Warning:

  • If you want feedback or review, you must support the project on GitHub:

Alternatively, you can still submit your PR to join the list of answered challenges or to be reviewed by a community member. 馃敟

@github-actions github-actions bot added 52 lazy component answer answer sponsor sponsor the project to be reviewed PR requests a review labels May 23, 2024
@tomalaforge tomalaforge added invalid solution is not working correctly and removed answer answer sponsor sponsor the project to be reviewed PR requests a review 52 lazy component labels May 23, 2024
Add author file
Remove thomas from the contributor list
Add a more direct hint
@LMFinney LMFinney force-pushed the angular/52-lazy-load-component branch from 52851e6 to 23ab602 Compare May 23, 2024 15:12
Update "Go to the latest Challenge" link
Lazy load via defer
The visual effect of deferral will work if AppComponent and/or TopComponent are NgModule-based.
However, in order for lazy loading to happen, both need to be standalone.
@LMFinney LMFinney force-pushed the angular/52-lazy-load-component branch from 23ab602 to a3cca63 Compare May 23, 2024 16:05
@tomalaforge tomalaforge added answer author answer of author sponsor sponsor the project and removed invalid solution is not working correctly labels May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
answer author answer of author sponsor sponsor the project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants