search

dfilatov / vidom

Library to build UI based on virtual DOM
MIT License
415 stars 16 forks source link

Deprecate `isMounted` #162

Closed himalayafan closed 8 years ago

himalayafan commented 8 years ago

https://github.com/facebook/react/issues/5465

dfilatov commented 8 years ago

I've read link you gave but can't understand what alternative is supposed.

himalayafan commented 8 years ago

Hey. It seems to me that they are going to use promises. I didn't find link for this, but I was reading last week in the React documentation an how-to-do this.

himalayafan commented 8 years ago

Ah sorry. Found some of the info now: https://github.com/facebook/react/pull/5587

dfilatov commented 8 years ago

It doesn't look like a solid solution. A workaround with using cancellablePromise works only for the first step. If you have chain of promises it won't work.