SofPyon / inaka-portal

[DEPRECATED] inaka-portal は PortalDots に改称し、リポジトリも移転します。移行先➡️

https://github.com/portal-dots/PortalDots

MIT License

5 stars 0 forks source link

【エラー画面】ページデザインを他のページと統一する #187 #277

Closed hosakou closed 4 years ago

hosakou commented 4 years ago

実装内容

close #187

エラー画面のデザインを変更
- v2のドロワーなどを表示するように(503以外)
500 503 のみ Twitter を表示するようにしてみました
- .env の PORTAL_ADMIN_TWITTER に Twitterスクリーンネームの @ 以下を指定すると表示されます

懸念点

レビュワーに見て欲しい点

デザイン的にどうなのか

参考URL

codecov[bot] commented 4 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@92137b2). Click here to learn what that means. The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #277   +/-   ##
=========================================
  Coverage          ?   24.98%           
  Complexity        ?      447           
=========================================
  Files             ?      112           
  Lines             ?     1353           
  Branches          ?        0           
=========================================
  Hits              ?      338           
  Misses            ?     1015           
  Partials          ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 92137b2...b6293b2. Read the comment docs.

SofPyon commented 4 years ago

あ、ごめん！ wip ついてたのにレビューしちゃった

hosakou commented 4 years ago

@SofPyon

大丈夫です！実はアドバイスほしかったのでちょっと期待してました！

hosakou commented 4 years ago

@SofPyon 修正しました！

hosakou commented 4 years ago

同じエラー文言が2箇所に出てくるのは微妙なので

ぼくもちょっと思ってました！

3点修正しました確認お願いします！