Created
August 11, 2018 01:19
-
-
Save kohei-takata/061fb58174d902e2d0427def17bc725d to your computer and use it in GitHub Desktop.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[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アプリを作成し、再現することを確認する。 |
↑コメントした
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
似たエラーを再現した。
Optionalな引数は順序が保証されていないと例外を吐くことを確認した。
第二引数に文字列以外がきた場合、
console.warn
で適切なワーニングメッセージを出すべきでは?https://github.com/facebook/react-native/blob/d01ab66b47a173a62eef6261e2415f0619fefcbb/Libraries/Alert/AlertIOS.js#L89
↑のalertのOptional引数で型が合ってない場合には適切なエラーを出した方がいい。
もしよかっら修正しましょうか?