Skip to content
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.

Sign up for GitHub

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

Jump to bottom

メッセージ機能を実装 #592

Open
wants to merge 53 commits into
base: main
Choose a base branch
from
Open

メッセージ機能を実装 #592

wants to merge 53 commits into from

Conversation

rito528
Copy link
Member

@rito528 rito528 commented Nov 1, 2024 •
edited
Loading

主な変更

  • Message に関する実装を追加
    • Message の作成
    • Message の取得
    • Message の編集
    • Message の削除
  • AuthorizationGuard (データに依存したアクセス権をドメインとして落とし込むため)の実装

@rito528
feat: メッセージ機能用テーブルの定義
5f38933
@rito528
feat: メッセージ機能の構造体を定義
9f7e83f
@rito528
feat: メッセージの送信を受け付ける機能の実装
749f891
@rito528
test: メッセージのテストを追加
ed6bf1b
@rito528
fix: messages テーブルを作成する SQL 文の構文エラーを修正
1f9c8ee
@rito528
feat: 権限がないメッセージを送信した時のレスポンスを実装
19ca99e
@rito528
feat: Message struct を再構築する関数の定義
e6f16bd
@rito528
chore: rustfmt の設定を変更
334ece2
@rito528
feat: Message を取得するエンドポイントを実装
426de76
@rito528
refactor: Message Struct からメッセージ取得ハンドラレスポンスへの依存を剥がす
674f903
@rito528
refactor: Message を作成できるか確認する関数を切り出す
32477bc
@rito528
feat: ドメインモデルの操作権限の明示することができるデータ型の定義
aa3270e
@rito528
test: AuthorizationGuard のテストを追加
653b4ae
@rito528
fix: AuthorizationGuard で受け取る T 型にトレイト境界を設定する
d7b0e7f
@rito528
refactor: 引数名の変更
3c37d82
@rito528
docs: AuhorizationGuard のドキュメントを書く
6de0f83
@rito528
refactor: post_message_handler から resolve 関数を呼び出さない
9128a50
@rito528
feat: AuthorizationGuard に Delete 操作についてを実装
52e7238
@rito528
docs: AuthorizationGuard の実装について説明を追加
87cf2a6
@rito528
fix: AuthorizationGuard の説明の脱字を修正
76d066a
@rito528
test: AuthorizationGuardDefinitons のドキュメントテストを使用例に即した形に変更
7ccd626
@rito528
chore: get_message -> get_messages
f12e9aa
@rito528
fix: compile error
0da2372
@rito528
refactor: message 周りをフォームコンポーネントに移動
4c8d1b5
@rito528
feat: メッセージを削除するハンドラの実装
60a0e5a
@rito528
refactor: fetch_messages_by_answer_id -> fetch_messages_by_answer
51b51a4
@rito528
refactor: message_posted_user -> message_sender
511e74f
@rito528
chore: tracing::instrument のつけ忘れを直す
271ce28
@rito528
fix: メッセージの削除権限の変更
6497238
@rito528
feat: authorization_guard に update action を追加
baddb47
@rito528
chore: proptest_async を使っていないので削除
82c5d72
@rito528
fix: Message の作成権限を変更
3674ed5
@rito528
refactor: Message の権限定義の意図をより明確にする
e71548b
@rito528
docs: Message 操作の権限定義に操作可能な条件の説明を書く
d5fac25
@rito528
docs: Message の can_update 関数に Administrator による更新の許可をしない理由を明記
4549bcd
@rito528
docs: Message の can_delete 関数に Administrator による Message の削除を許可しない理由を明記
9365b3c
@rito528
test: 不要になったテストを削除
fc64820
@rito528
refactor: form_raw_parts から unsafe を外す
cfea0d8
@rito528
chore: /messages/:message_id の PATCH, DELETE メソッドを一般公開エンドポイントに追加
dee8b6e
@rito528
fix: post message handler が返すレスポンスを API 定義に合わせる
0be1c01
@rito528
test: authorization_guard の update と delete に関するテストを追加
0da7b57
@rito528
fix: メッセージ用 API の URL を変更
340fdbf
@rito528
fix: post_message_handler で answer_id を受け取るようにする
aff32da
@rito528
fix: messages endopoint path
6987cc5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@loliGothicK loliGothicK Awaiting requested review from loliGothicK loliGothicK is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@rito528