Skip to content

Instantly share code, notes, and snippets.

@kohei-takata
Created August 11, 2018 01:19
Show Gist options
  • Save kohei-takata/061fb58174d902e2d0427def17bc725d to your computer and use it in GitHub Desktop.
Save kohei-takata/061fb58174d902e2d0427def17bc725d to your computer and use it in GitHub Desktop.
[Issues · facebook/react-native · GitHub](https://github.com/facebook/react-native/issues?q=is%3Aopen+is%3Aissue+label%3A%22Good+first+issue%22)
good first issueからできそうなやつを探す
上から見ていく
metroわからんからむり
[Update new project template to metro-react-native-babel-preset · Issue #20327 · facebook/react-native · GitHub](https://github.com/facebook/react-native/issues/20327)
よくわからんcloseされてるからむり
[Validate Environment Scripts · Issue #19694 · facebook/react-native · GitHub](https://github.com/facebook/react-native/issues/19694)
いけそう
[Alert.alert without message kills apps even after refreshing · Issue #18439 · facebook/react-native · GitHub](https://github.com/facebook/react-native/issues/18439)
Alertのメソッドの使い方が間違っているぽい
(引数が足りていないだけ)
ただ、エラーメッセージは親切とは言えないので、そこを修正できるかもしれない。
中の人の意見も聞きたいので、issueにコメントしておき、返信を待ってから対応するとスムーズそう
引数が足りていない(順番が違う)ときに、エラーになることを確認したい。
まず、最小構成のRNアプリを作成し、再現することを確認する。
@binaryta
Copy link

binaryta commented Aug 11, 2018

似たエラーを再現した。
image

引数が足りていない(順番が違う)ときに、エラーになることを確認したい。

Optionalな引数は順序が保証されていないと例外を吐くことを確認した。

第二引数に文字列以外がきた場合、console.warn で適切なワーニングメッセージを出すべきでは?
https://github.com/facebook/react-native/blob/d01ab66b47a173a62eef6261e2415f0619fefcbb/Libraries/Alert/AlertIOS.js#L89
↑のalertのOptional引数で型が合ってない場合には適切なエラーを出した方がいい。
もしよかっら修正しましょうか?

@kohei-takata
Copy link
Author

kohei-takata commented Aug 11, 2018

↑コメントした
facebook/react-native#18439 (comment)

次これやっていく
facebook/react-native#16228

このコメントのとおりにやっていく
facebook/react-native#16228 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment