OwnerGuardのリファクタリング #65

Z00One · 2024-04-17T11:37:49Z

🔍 このPRで解決したい問題は何ですか？

実装した機能や解決した課題、PRを開くきっかけや目的をReviewerが理解しやすいように書いてください。
バックログのリンク、ダイアグラム、Figmaなどの関連ドキュメントを添付するとより良いです。

リソースに接近する時の権限確認機能の修正

✨ このPRで主に変わったことは何ですか？

問題を解決しながら、主な変更点を書いてください。

OwnerGuardのリファクタリング

🔖 主な変更点以外に追加で変更された部分はありますか？

ない場合は「なし」と記載してください。

クイズのセットに関する機能の修正
マテリアルに関する機能の修正

🙏🏻 Reviewerに特に見ていただきたい部分はありますか？

開発過程で他の方の意見が気になったり、クロスチェックが必要だと感じたコードがあれば教えてください。

なし

🩺 このPRでテストや検証が必要な部分はありますか？

テストが必要な項目やテストコードが追加されたら一緒に書いてください。

なし

📚 関連するIssueやTrello、ドキュメント

このPRが解決しようとしている問題に関連するIssueやドキュメント、Trelloがあればリンクを記載してください。Trelloの場合タイトルの形で添付してください。

権限関連機能の修正 #64

🖥 作動する様子

スクリーンショットや録画したビデオ、またはgifを追加して、Reviewerが変更点を理解するのに役立ててください。

📌 PRを行う際の注意点

Reviewerはコードレビュー時に良いコードの方向性を示しますが、コード修正を強制することはありません。
Reviewerは良いコードを見つけた場合、賞賛と励ましを惜しみません。
レビューは特別なケースでない限り、Reviewerに指定された時点から3日以内に行ってください。
コメント作成時にPrefixにP1、P2、P3を書いていただくと、Assigneeがより明確にコメントに対して対応することができます。
- P1 : 必ず反映してください (Request Changes) - 問題が発生したり、脆弱性が発見されたケースなど。
- P2 : 反映を積極的に検討していただければと思います (コメント)。
- P3 : こんな方法もあるんじゃないかな～などの些細な意見です (Chore)。

…e method YJU-OKURA#64

…ith Prisma.ModelName

Added a check to throw a NotFoundException when trying to delete a material that does not exist.

-Changed the method to use `setQuiz.findUnique` instead of `quizList.findMany` - Updated the select fields to include `deadline` and `quizLists` - Adjusted the nested select fields to include `q_id` and `content` for quizzes

- Refactor the setQuizService to return a more structured response with deadline and quizList. - Added u_id parameter to checkBeforeMark function for better validation. - Updated ConflictException to GoneException for deadline passed scenario.

src/common/decorators/metadata.decorator.ts

src/common/decorators/use-owner-guards.decorator.ts

src/common/decorators/use-role-or-owner-guards.decorator.ts

src/common/guards/owner.guard.ts

src/modules/material/entity/material.entity.ts

yuminn-k

고생하셨습니다! :)

Z00One added 10 commits April 17, 2024 19:49

♻️ refactor: Refactor checkOwnership method and add getUIdFromResourc…

c4f49b1

…e method YJU-OKURA#64

🐛 fix: Remove unused ID_PARAM from metadata decorator

be141b0

🐛 fix: Fix description in UseOwnerGuards decorator

aac12c1

🐛 fix: remove id parameter

115e306

♻️ refactor: Removed the hard-coded string 'Prompt' and replaced it w…

8c36641

…ith Prisma.ModelName

🐛 fix: Remove unused import

20b70a7

✨ feat: Add file object to material entity

c2cf9df

🐛 fix: handle error when material not found

b6a46db

Added a check to throw a NotFoundException when trying to delete a material that does not exist.

🐛 fix: Fix getSettedQuizzesByMId method

bfceb28

-Changed the method to use `setQuiz.findUnique` instead of `quizList.findMany` - Updated the select fields to include `deadline` and `quizLists` - Adjusted the nested select fields to include `q_id` and `content` for quizzes

♻️ refactor: Refactor setQuizService

f0fcfad

- Refactor the setQuizService to return a more structured response with deadline and quizList. - Added u_id parameter to checkBeforeMark function for better validation. - Updated ConflictException to GoneException for deadline passed scenario.

Z00One added 🔨 Refactor 코드 리팩토링 🐞 BugFix Something isn't working 📬 API 서버 API 통신 🆗 safe CI workflow 동작 관련 labels Apr 17, 2024