shinonome-inc / mobile_sakai

0 stars 1 forks source link

模擬案件#7 ErrorPage実装まで #35

Closed yasyun closed 2 years ago

yasyun commented 2 years ago
yasyun commented 2 years ago
yasyun commented 2 years ago

変更点が思っていたよりも多くなり、コードも広範囲に差分が生まれてしまいそうなので、いったんFeedPage, TagPage, MyPageの3画面のみでPull Requestを提出し、LGTMをいただいてからOAuthPage, FollowPage, FeedDetailPage, TagDetailPage, ArticlePageのErrorViewを別で実装してレビューをいただこうと思う。

yasyun commented 2 years ago

Storyboard上にUIViewContainer Viewを配置してNetworkErrorViewやNotLoginPageViewControllerを表示していたが、UIViewをコードでaddSubView()して表示する仕様に変更した。したがって、isHidden()ではなくremoveFromSuperView()を使うように変更した。 現在はFeedPageMyPageの変更をしたので、OKをいただいてから後程TagPageも変更する。

yasyun commented 2 years ago

timeoutのErrorが発生した際にもerrorViewを表示する様に変更。 これでnetwork conditionerを使用した時にも、少し待ってtimeoutすればerrorViewが表示される。