模擬案件_#1_addQiitaOAuthPage

shinonome-inc / mobile_sakai

0 stars 1 forks source link

Closed yasyun closed 3 years ago

yasyun commented 3 years ago

・Issue

・レビューして欲しい点

アプリ起動からQiitaOAuth認証までの流れ(QiitaOAuthPageViewController.swiftの中身) アクセストークンはまだレスポンスをprintで表示しているだけなので、アクセストークン利用の点は飛ばしてください。

ShuheiYoshidaJP commented 3 years ago

1点目 OAuth以外にも実装されている部分が見受けられるので、レビューして欲しい点を具体的に示してもらえますでしょうかイメージ

2点目実装過程や参考にしたサイトなどを参照しながらレビューしたいので issueなどに残してください。起票されたissueは1点目と同じところに記載してください。

ShuheiYoshidaJP commented 3 years ago

SecretKey.swiftファイルをなんらかの形でレビュワーに渡せるような仕組みを作ってもらえるでしょうか。

mcz9mm commented 3 years ago

私にもください〜

SecretKey.swiftファイルをなんらかの形でレビュワーに渡せるような仕組みを作ってもらえるでしょうか。

mcz9mm commented 3 years ago

別issueで対応する場合、差分がでないので今のうちに修正が必要な箇所は個人的にメモなどしておくようにしておいてください。

mcz9mm commented 3 years ago

各コメントの流れがわかりにくい場合はFiles Changedから該当箇所のコードを確認すると経緯がわかるかと思います。

yasyun commented 3 years ago

先日、「修正内容のコメントが確認できないです」ということをPGritでお伝えした件なのですが、僕の方のGithubでコメントが多くなってきたことにより、自動的に途中のコメントがタブのようなものに隠れてしまっていたようで、気づけていませんでした。確認不足で申し訳ありませんでした。以後気をつけます。

mcz9mm commented 3 years ago

2点修正したらOKです。