Missing subcriptions management on auth component
josecarballosa opened this issue · 1 comments
josecarballosa commented
Could the missing subscription management on the auth component create some leaking problems? Is it OK to do it like this in a real-world app?
checkout this line:
and also again few lines below:
peterblazejewicz commented
@carballosa
Sometimes that is OK to not unsubscribe (where the source of the observable will relinquish the resource).
I've created a PR to put a context on this:
#153
The note here explains this:
https://angular.io/guide/router#observable-parammap-and-component-reuse
Also check Ben Lash comments from Ng Conf closing panel:
https://www.youtube.com/watch?v=PjHIqx0w4do
Cheers!