-
Notifications
You must be signed in to change notification settings - Fork 659
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat:管理画面へメンテナンスモードを通知する #6026
feat:管理画面へメンテナンスモードを通知する #6026
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Issueにあるように、プラグイン操作しているときに自動的にメンテナンスモードになり、気づかずに放置してしまうので、トップページだけではなくどの管理画面でも表示できるようにした方が良いかと思います。
@kiy0taka |
Codecov Report
❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more. @@ Coverage Diff @@
## 4.2 #6026 +/- ##
============================================
- Coverage 82.56% 82.56% -0.01%
Complexity 6425 6425
============================================
Files 475 475
Lines 25855 25860 +5
============================================
+ Hits 21347 21351 +4
- Misses 4508 4509 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
|
33a2365
to
311768c
Compare
311768c
to
5d0424d
Compare
動作確認で問題無ければ、次回VersionUP時に取り込む予定 |
概要(Overview・Refs Issue)
管理画面からメンテナンスモードか分かるようにしました。
関連Issue
メンテナンスモードであることを管理画面ですぐに確認できる #4271
方針(Policy)
「管理画面URLは、セキュリティのため推測されにくいものを設定してください」の実装を参考
(Controller側では判定のみ、templateで出し分けしている)
実装に関する補足(Appendix)
テスト(Test)
日本語ver
英語ver
相談(Discussion)
マイナーバージョン互換性保持のための制限事項チェックリスト
レビュワー確認項目