Thanks! To be on the safe side `.complete();` can be added.
In the exact case as in the example all subscriptions to the `componentDestroyed` subject will end with the first emission/`.next();` call, because this subject is used as the source of the`takeUntil` operator, so it wouldn't have any subscriptions made to it afterwards and there will be no references to anything external, so there shouldn't be any memory leaks/etc.