Discussion on: The Ultimate Answer To The Very Common Angular Question: subscribe() vs | async Pipe

View post

Replies for: You can also do this.fooExists$ = this.store.pipe(select(foo), filter(foo => !!foo)); then you can also this.fooExists$.subscribe(operation). Bu...

Would i need to manually unsubscribe in OnDestroy if I did it this way (not @Effect) as you have described?

Yes, whenever you do explicit .subscribe() call, you are automatically responsible to call .unsubscribe(). Even better would be to use .pipe(takeUntil(onDestroy$)) which is declarative ;)

Stephen Chiang • Oct 8 '18 • Edited

awesome, thanks!

so then I would also need to do this, right?

ngOnDestroy() {
    this.onDestroy$.next(true);
    this.onDestroy$.unsubscribe();
  }

Tomas Trajan 🇨🇭 • Oct 8 '18 • Edited

ngOnInit() {
   this.source$.pipe(takeUntil(onDestroy$)).subscribe()
}

ngOnDestroy() {
    this.onDestroy$.next(true);
    this.onDestroy$.complete()
}

CHeers!