connected-react-routerを使ってurlを変えたのに表示されるコンポーネントが変わらない【React&Redux&TypeScript】

Question

前提・実現したいこと

/signinページでボタンを押すと/countページに移動する

↓

Countは今のところh1要素をレンダリングしているだけです。

発生している問題・エラーメッセージ

サインインページのログインボタンを押すと問題なく/countのurlに変わるのですが、表示されるコンポーネントが変わりません。しかしブラウザで再読み込みすればCountコンポーネントが表示されます

import * as React from 'react'
import * as ReactDOM from 'react-dom'
import { Provider } from "react-redux"
import App from './App'
import { store, history } from './modules/Store'


ReactDOM.render(
  <Provider store={store}>
    <App history={history}/>
  </Provider>,
  document.getElementById('app')
);

ルートディレクトリ下のindex.jsではAppコンポーネントをレンダリングするだけ。

import * as React from 'react'
import { History } from 'history'
import { BrowserRouter, Route } from "react-router-dom";
import { ConnectedRouter } from 'connected-react-router'
import SignIn from './containers/Signin'
import Count from './containers/counter/Header'

type AppProps = {
  history: History;
}

const App = ({ history }: AppProps) => {
    return(
      <ConnectedRouter history={history}>
        <BrowserRouter>
          <Route exact path='/signin' component={SignIn} />
          <Route path='/count' component={Count} />
        </BrowserRouter>
      </ConnectedRouter>
    )
}


export default App

Appコンポーネントはurlが/signinの時にSigninコンポーネントを、/countの時にCountコンポーネントをレンダリングします。

import * as React from 'react';
import { connect } from 'react-redux'
import { logInIfFetchUser } from '../modules/ActionCreater'
import { AppState } from '../modules/Reducers'


type SignInProps = {
  logInIfFetchUser: any
}
class SignIn extends React.Component<SignInProps> {
  render(){
    let email: any
    let password: any

    return(
      <div>
        <h1>サインイン</h1>
        <form onSubmit={(e) => {
          e.preventDefault()
          console.log(`searching user with email: ${email.value}, password: ${password.value}`)
          this.props.logInIfFetchUser(email.value, password.value)
        }}>
          <input type='text' ref={node => email = node}/>
          <input type='password' ref={node => password = node}/>
          <button type='submit'>ログイン</button>
        </form>
      </div>
    )
  }
}

const mapStateToProps = (state: AppState) => {

}

export default connect(mapStateToProps,{ logInIfFetchUser })(SignIn)

SignInコンポーネントです。
onSubmitした時のアクションが以下の通りです。

import axios from 'axios'
import { Action } from 'redux'
import { ThunkAction } from "redux-thunk"
import { push } from 'connected-react-router'
import { LOGIN } from './ActionTypes'
import { AppState } from './Reducers'

export function logIn(userName: string) {
  return {
    type: 'LOGIN' as typeof LOGIN,
    name: userName
  }
}


export const logInIfFetchUser = (
  email: string,
  password: string
): ThunkAction<void, AppState, null, Action<string>> => async dispatch => {
  axios.get(`http://localhost:8080/api/?email=${email}&password=${password}`)
    .then((response) => {
      dispatch(logIn(response.data[0].name))
      dispatch(push('/count'))
    })
}

SignInで入力されたアドレスとパスワードと一致するユーザーをapiサーバーから返します。
その後、/countに移動します。
リクエストは成功し、ユーザーが返ってきたことは確認しました。

試したこと

connect()(SignIn)をさらにwithRouter()でラップしてみました。

connect()の引数を以下のように変えてみました。

export default connect(mapStateToProps, { logInIfFetchUser }, null, {
  pure: false
})(SignIn)

その他のファイル(ReducerやStoreなど)

import { Action, LOGIN } from './ActionTypes'
import { Reducer, combineReducers } from 'redux'
import { History } from 'history'
import { connectRouter } from 'connected-react-router'

type userReducerState = {
  name: string
}

const initialState = {
  name: ''
}

export const userReducer: Reducer<userReducerState, Action> = (state = initialState, action) => {
  switch(action.type) {
    case LOGIN:
      return {
        name: action.name
      }
    default:
      return state;
  }
}

export type userReducer = typeof userReducer

export const rootReducer = (history: History) => combineReducers({
  router: connectRouter(history),
  userReducer
})

export type AppState = ReturnType<typeof rootReducer>

import { createStore } from 'redux'
import { applyMiddleware } from "redux";
import thunk from 'redux-thunk';
import { createBrowserHistory } from 'history'
import { routerMiddleware } from 'connected-react-router'
import { rootReducer } from "./Reducers";

export const history = createBrowserHistory()

export const store = createStore(
  rootReducer(history),
  applyMiddleware(
    routerMiddleware(history),
    thunk
  )
)

import { logIn } from './ActionCreater'

export const LOGIN = 'LOGIN'
export type Action = ReturnType<typeof logIn>

userReducerはまだ使っていません。後にユーザー名を取り出すつもりです。

Answer 1

こちら、connected-react-routerのissueにも上がっている現象のようですね。
APIコールの後にpushをdispatchしても、URLは変わるがコンポーネントはそのままとのことです。
(#230)

パッと思いつく代替案としては、ログイン中か否かのStateを作って dispatch(push('/count')) の代わりにログイン中フラグをON→コンポーネント側の componentDidUpdate 等でログイン中フラグがOFF→ONになったら history.push('/count') 等で画面遷移させる、で実現できると思います。

もしくは、redux-sagaを使った例ですが丁寧なサンプルも落ちていたので併せて確認してみてください。
Reactサンプル：redux-sagaとreact-router v4でWebAPIの結果に応じて画面遷移させる。

Comments

[southernX]

history.pushのhistoryはcreateBrowserHistoryで作ったものですか？mapStateToPropsがうまくいかなかったのでアクション内で実行したところ依然としてコンポーネントが変わらないままです。

[guissy-k]

はい、createBrowserHistoryで作ったhistoryを想定していましたが、それだと動かないのですね・・・失礼しました。

それでは、下記を参考にconnected-react-routerの push をpropsに設定して props.push('/count') ではどうですか？
参考：https://qiita.com/hiroya8649/items/34979f2008cf92c110ff#with-react-redux

[southernX]

propに変えてみましたが解決しませんでした・・・
参考を拝見しましたが、おそらくこのpropsはconnect()でpushを渡したからコンポーネント内でできるのだと思います。

[southernX]

Appコンポーネント内の<BrowserRouter></BrowserRouter>を<Switch>に変えると正しくページ遷移出来ました！回答ありがとうございました。

[guissy-k]

解決できたようで良かったです。あまりお力になれずすみませんでした。