Skip to content

Angular upgrade to v16#183

Merged
jggoebel merged 60 commits intohobbyfarm:masterfrom
svalabs:ng-version-upgrade
Mar 8, 2024
Merged

Angular upgrade to v16#183
jggoebel merged 60 commits intohobbyfarm:masterfrom
svalabs:ng-version-upgrade

Conversation

@PhilipAB
Copy link
Copy Markdown
Contributor

@PhilipAB PhilipAB commented Sep 26, 2023

What this PR does / why we need it:
This PR updates the Angular version from v12 to v16.

fixes hobbyfarm/hobbyfarm#301

PhilipAB added 26 commits March 3, 2024 22:56
Category form can become null during runtime. Hence let's make it nullable during compile time, too.
If the observable does not emit any value and completes directly, using first() results in an error.

Using take(1) generally does the same as first() but with a minor difference.

take(1) mirrors the empty source observable in case it is empty and then also completes instead of throwing an error as first() does.
Otherwise the alerts cannot be closed via two-way-binding...
Advantages: ngAfterViewInit is only called once. So, we only need to manage one subscription.
@PhilipAB PhilipAB marked this pull request as ready for review March 8, 2024 02:53
@jggoebel jggoebel merged commit d69f1dd into hobbyfarm:master Mar 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update angular to v15

2 participants