SoftwareFoundationGroupAtKyotoU / automata

Other
3 stars 10 forks source link

isMounted && setState の Mixin 化 #329

Open skymountain opened 9 years ago

skymountain commented 9 years ago
if (this.isMounted()) {
    this.setState( ... );
}

というパターンも頻出なので,これをまとめた関数を提供を mixin を作成する.

westpaddy commented 9 years ago

これ今、アンマウントされてるときにsetStateすると例外(エラー)が投げられますけど、次期バージョン(というかすでにリリースされてるv0.13)では警告が出るだけに変わるので、(使用するReactのバージョンを上げるなら)ガードを外すだけでいいと思います。

skymountain commented 9 years ago

上げましょう.

westpaddy commented 9 years ago

breaking changesがあるのでさくっと上げると死にそうです。