Grading-X / Grading-X-BE

http://3.34.49.173:8080/swagger-ui/index.html#/
0 stars 0 forks source link

공지사항 기능 추가 #44

Closed chaejm55 closed 5 months ago

chaejm55 commented 5 months ago

course 하위 V2 기능으로 공지사항 기능을 추가했습니다.

github-actions[bot] commented 5 months ago

Test Results

1 tests   1 :white_check_mark:  0s :stopwatch: 1 suites  0 :zzz: 1 files    0 :x:

Results for commit 0973dc52.

:recycle: This comment has been updated with latest results.

chaejm55 commented 5 months ago

사실 CourseId를 받고 있지만 보다보니까 생성을 제외하고는 CouseId를 굳이 받을 필요가 있나? 싶긴 합니다. 그리고 조건을 생각하면 CourseId에 해당하는지 Check 하기보다는 수정, 삭제시 요청한 사용자가 해당하는 공지사항의 작성자가 맞는지, 생성의 경우에는 Course의 담당자가 맞는지 체크하는 정도가 필요하다고 생각되네요

음 일단 courseId를 받긴하다보니 넣었는데 아예 빼는게 훨씬 좋을까요?

sim-mer commented 5 months ago

생성 빼고는 딱히 courseId가 필요해 보이진 않네요