Skip to content

Conversation

@KaichiManabe
Copy link
Contributor

@KaichiManabe KaichiManabe commented Dec 15, 2024

PRの概要

close #298

具体的な変更内容

管理者画面を作ろかと思ったが、管理者画面を作ってメンテナンスするコストを考えると、とりあえず直書き運用でいい気がした #540

影響範囲

動作要件

補足

レビューリクエストを出す前にチェック!

  • 改めてセルフレビューしたか
  • 手動での動作検証を行ったか
  • server の機能追加ならば、テストを書いたか
    • 理由: 書いた | server の機能追加ではない
  • 間違った使い方が存在するならば、それのドキュメントをコメントで書いたか
    • 理由: 書いた | 間違った使い方は存在しない
  • わかりやすいPRになっているか

@KaichiManabe KaichiManabe merged commit 4eeb364 into main Dec 15, 2024
8 checks passed
@KaichiManabe KaichiManabe deleted the add-notification branch December 15, 2024 10:08
nakaterm pushed a commit that referenced this pull request Mar 6, 2025
# PRの概要
close #298 

## 具体的な変更内容
管理者画面を作ろかと思ったが、管理者画面を作ってメンテナンスするコストを考えると、とりあえず直書き運用でいい気がした #540 

## 影響範囲

## 動作要件

## 補足

## レビューリクエストを出す前にチェック!

- [ ] 改めてセルフレビューしたか
- [ ] 手動での動作検証を行ったか
- [ ] server の機能追加ならば、テストを書いたか
  - 理由: 書いた | server の機能追加ではない
- [ ] 間違った使い方が存在するならば、それのドキュメントをコメントで書いたか
  - 理由: 書いた | 間違った使い方は存在しない
- [ ] わかりやすいPRになっているか

<!-- レビューリクエスト後は、Slackでもメンションしてお願いすることを推奨します。 -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

グローバルの「お知らせ」的なページ 締切: 12/15

3 participants