-
Notifications
You must be signed in to change notification settings - Fork 95
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Angular.jsからReactへ移行する #134
Conversation
|
||
_didChannelIdChange: -> | ||
@currentChannelId != @newChannelId && | ||
!(@currentChannelId == '' && @newChannelId == 'no_select') # 未選択状態で「ステータス」を選択してなかったら |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line exceeds maximum allowed length
SpinnerStore.addChangeListener(@_onChange) | ||
|
||
componentWillUnmount: -> | ||
SpinnerStore.removeChangeListener(@_onChange); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line contains a trailing semicolon
`<div className='work-episodes container'> | ||
<h2 className='text-center'>エピソード</h2> | ||
<div className='multiple-checkin'> | ||
<div className={switchButtonClass} onClick={this.startMultipleCheckinMode} onTouchStart={this.startMultipleCheckinMode}>一括チェックイン</div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line exceeds maximum allowed length
Conflicts: app/controllers/checkins_controller.rb
render: -> | ||
props = @props | ||
|
||
checkinPath = "/works/#{props.workId}/episodes/#{props.episodeId}/checkins/#{props.checkin.id}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line exceeds maximum allowed length
しばらくAngular.jsで良いや、という気持ちなりました。
このプルリクエストの中に副産物的に機能を改修した部分があるので、それをmasterブランチに取り込み、このプルリクエストは一旦破棄したいと思います。 |
という条件でJSのライブラリ(フレームワーク)を探したところ、Reactが一番フィットした気がしたので、移行します。