感想

[yamachu]

treasure_front_app/src/App.js

Line 56 in 3be6198

<Router>

Nice Router!

https://github.com/nasa-playground/treasure_front_app/blob/master/src/pages/index_article.js
一覧表示も実装してある！いいね！

https://github.com/nasa-playground/treasure_front_app/blob/master/src/pages/index_article.js#L27
削除済みだとアクセス出来ないからリストのリフレッシュのためにReloadしてるんだね、良さそう。

けど
https://github.com/nasa-playground/treasure_front_app/blob/master/src/pages/index_article.js#L11
https://github.com/nasa-playground/treasure_front_app/blob/master/src/api.js#L19
引数がundefinedだから一応動作してるけれども、引数の数はちゃんとあわせたいね

treasure_front_app/src/pages/update_article.js

Lines 22 to 23 in 3be6198

	<input type="text" value={title} onChange={(e) => setTitle(e.target.value)}/>
	<textarea value={body} onChange={(e) => setBody(e.target.value)} />

add_articleと同じようなformを使うのであれば、共通化とかも考えてみて良いかもね。

あと細かい話だけど、

treasure_front_app/src/App.js

Lines 42 to 48 in 3be6198

	{user === null ? (
	<button onClick={firebase.login}>Please login</button>
	) : null}
	<div>{message}</div>
	<p>{errorMessage}</p>
	<button onClick={() => getPrivateMess()}>Get Private Message</button>
	<button onClick={firebase.logout}>Logout</button>

ログインしていないとボタンを消すだけじゃなくて、ログイン必須の他の項目も消しちゃって良いかもね

全体的に良さそう:+1: