[FEAT] Notice api 일부 구현 (create, get, update, delete) #12

shj1081 · 2024-07-19T16:07:52Z

작성자: @shj1081

#9

체크 리스트

적절한 제목으로 수정했나요?
상단에 이슈 번호를 기입했나요?
Target Branch를 올바르게 설정했나요?
Reviewers/Assignees/Labels을 알맞게 설정했나요?

작업 내역

Notice API 일부 구현 (DTO, Repository, Service, ServiceImpl, Controller)

createNotice API
getNotice API
getNoticeList API
updateNotice API
deleteNotice API

BaseNoticeEntity, BaseTimeEntity, Notice/EventNotice Entity 수정

builder pattern 을 위한 @NoArgsConstructer, @SuperBuilder annotation 추가
updateNotice 를 위한 Setter method 추가

비고

적당히 작업 브랜치를 나누기가 헷갈려서 일단 시험삼아 올려보았습니다... 🫠
전체적인 방향성이 올바른지도 봐주시면 정말 감사드리겠습니다 🙇🏻🙇🏻
추가로 기능 단위의 완전 구현 후에 PR 을 올려야 하는지 부분 구현 후에 PR 을 올려도 되는지도 여쭙고 싶습니다

builder 추가 과정에서 global domain 을 일부 수정하였는데, 문제 없는지 궁금합니다
attached files 를 위한 uuid field 미구현
create / delete 시 Admin 인지 확인 미구현
- 이 경우 로그인이 필요한 것인지 아닌지 구분이 잘 가지않아 일단은 보류했습니다

작업 branch 2 예정

getNotice hitCount 증가 logic 추가
updateNotice Setter 가 아닌 Repository 이용하는 방식으로 변경 (현준님 code 참고)
RestDocs test code 추가

yesjuhee

첫 PR 축하합니다 🎉
Entity, DTO 관련 리뷰입니다!

src/main/java/com/scg/stop/global/domain/BaseTimeEntity.java

src/main/java/com/scg/stop/global/domain/BaseNoticeEntity.java

src/main/java/com/scg/stop/domain/notice/domain/dto/NoticeDto.java

yesjuhee

컨트롤러, 서비스 리뷰입니다!

src/main/java/com/scg/stop/domain/notice/domain/controller/NoticeController.java

src/main/java/com/scg/stop/domain/notice/domain/service/NoticeServiceImpl.java

src/main/java/com/scg/stop/domain/notice/domain/controller/NoticeController.java

yesjuhee · 2024-07-20T05:33:52Z

이정도 분량의 PR이 좋은 것 같습니다 👍
controller, service, dto, repository 로 잘 나눠서 작성 됐네욥 여기에 test 코드를 이용한 restdocs 기능만 추가되면 될 것 같습니다
PR은 부분 구현 보다는 기능 단위로 들어가 있어야 리뷰어가 리뷰하기 좋을 것 같습니다. 이정도 PR이면 딱 좋습니다!
global domain 수정한 부분은 간단한 부분이라 따로 브랜치를 팔 필요는 없을 것 같습니다.

jcy0308 · 2024-07-21T14:45:53Z

주희님 리뷰이후로 커밋된게 혹시 있나요?? 혹시 로컬에 있으시면 푸시해주세요
내용이 좀많아서 리뷰정리되면 마저 리뷰하겠습니다

리뷰반영된 부분도 conversation은 닫아주세요!

shj1081 · 2024-07-21T14:48:05Z

주희님 리뷰이후로 커밋된게 혹시 있나요?? 혹시 로컬에 있으시면 푸시해주세요 내용이 좀많아서 리뷰정리되면 마저 리뷰하겠습니다

수정 사항 반영한 것 push 해두었습니다! 🧑🏻‍💻

jcy0308

저희 프로젝트는 저번에 얘기한대로 도메인형 구조로 정했는데,
그 구조에 맞게 패키지 구조도 수정해주시면 감사하겠습니다.
https://velog.io/@sunil1369/Spring-boot-%ED%8C%A8%ED%82%A4%EC%A7%80-%EA%B5%AC%EC%A1%B0

src/main/java/com/scg/stop/domain/notice/domain/dto/NoticeDto.java

src/main/java/com/scg/stop/domain/notice/domain/service/NoticeService.java

src/main/java/com/scg/stop/domain/notice/domain/dto/NoticeDto.java

src/main/java/com/scg/stop/domain/notice/domain/controller/NoticeController.java

src/main/java/com/scg/stop/global/domain/BaseTimeEntity.java

shj1081 · 2024-07-22T05:15:18Z

저희 프로젝트는 저번에 얘기한대로 도메인형 구조로 정했는데, 그 구조에 맞게 패키지 구조도 수정해주시면 감사하겠습니다. https://velog.io/@sunil1369/Spring-boot-%ED%8C%A8%ED%82%A4%EC%A7%80-%EA%B5%AC%EC%A1%B0

엇 패키지 구조가 잘못되있엇군요...
수정하도록 하겠습니다

5d037e0

shj1081 · 2024-07-22T06:12:40Z

다음과 같은 추가 수정 사항 반영 하고 push 하도록 하겠습니다 🧑🏻‍💻

dto inner class --> reponse, request dto 로 분리
builder pattern --> 정적 팩토리 method
update using setter --> update using repository
put api 의 response body 에 관련 값 포함하여 return

yesjuhee

리뷰가 너무 길어지는 것 같고 위의 논의들은 어느정도 마무리 된 것 같아서 해당 브랜치 머지 하고
추가 수정 사항은 새로운 PR로 올려주실 수 있나욥? 우선 Approve 하겠습니다!

yesjuhee · 2024-07-22T06:24:19Z

머지할 때 Squash 말고 Commit and merge로 하시면 됩니다!

shj1081 · 2024-07-22T06:34:05Z

리뷰가 너무 길어지는 것 같고 위의 논의들은 어느정도 마무리 된 것 같아서 해당 브랜치 머지 하고 추가 수정 사항은 새로운 PR로 올려주실 수 있나욥? 우선 Approve 하겠습니다!

알겠습니당
일단 merge 하고 다음 브랜치에서 작업 후 PR 올리겠습니다!

2tle

고생하셨습니다!

@authuser

* feat: talk 및 jobInterview 레포지토리 추가 * feat: jobInterview DTO 추가 * feat: add create and get jobinterview api * feat: add update and delete jobinterview api * fix: change BaseVideoEntity access level * refactor: add new line in JobInterview Category * refactor: change dto name * fix: change string check annotation to NotBlank * refactor: change category enum class name * feat: add enum validator * refactor: apply dto format discussed in PR #12 * fix: apply static factory method * chore: remove superbuilder annotation in BaseTimeEntity * chore: transactional 어노테이션 클래스로 이동 * chore: 엔티티 객체 생성 스타일 변경 * fix: pathVariable 명시 * fix: cascade 옵션 변경 * chore: dirty checking 사용하는 방법으로 update 변경 * feat: change exception to custom BadRequestException * chore: 접근제어자 제거 * feat: 잡페어 인터뷰 생성 테스트코드 작성 * feat: add JobInterview create Testcode * fix: ExceptionCode 추가 및 변경사항 대응 * feat: 잡페어 인터뷰 컨트롤러 테스트 * feat: 잡페어 인터뷰 RestDocs * feat: create QuizRepository * feat: add Talk Request/Response DTO * feat: implement talkservice and add exception code * feat: implement talkController * feat: implement TalkControllerTest * feat: implement TalkControllerTest * feat: 대담영상 api 문서 작성 * feat: 퀴즈 포함 대담 API 구현 * fix: 대담 연관관계 업데이트 버그 수정 * feat: add talker belonging and name field in jobinterview entity * feat: update jobinterview testcode * chore: remove status header * feat: remove HasQuiz And Apply JsonUnwrapped Annotation * refactor: remove Talks(List) Response DTO * feat: update TalkController TestCode * chore: update Talk API Docs * refactor: 엔티티에서 DTO 사용 제거 * feat: @authuser 적용 및 testcode 수정 * chore: add token to request cookie and header * fix: change local.yml * refactor: change video package location * refactor: test code 패키지 변경 * fix: change db url in local.yml

shj1081 added 5 commits July 20, 2024 00:20

feat: Add Notice,EventNotice entity Superbuilder, setter for update

6616d1f

feat: Implement NoticeDto Inner Static class

61384a0

feat: Implement NoticeRepository

594614c

feat: Implement NoticeService interface, NoticeServiceImpl

e656cec

feat: Implement NoticeController

26899e9

shj1081 added 🚀 Feature 기능 개발 🎯 API API 개발 관련 labels Jul 19, 2024

shj1081 requested review from 2tle, jcy0308, yeobi01, yesjuhee and simta1 July 19, 2024 16:07

shj1081 self-assigned this Jul 19, 2024

shj1081 linked an issue Jul 19, 2024 that may be closed by this pull request

🚀 [FEAT] Notice, EventNotice 관련 API 개발 #9

Closed

14 tasks

shj1081 removed a link to an issue Jul 19, 2024

🚀 [FEAT] Notice, EventNotice 관련 API 개발 #9

Closed

14 tasks

yesjuhee reviewed Jul 20, 2024

View reviewed changes

yesjuhee added the Notice Domain Notice 도메인 관련 label Jul 20, 2024

shj1081 added 4 commits July 21, 2024 23:50

fix: @NoArgsConstructor(access = PROTECTED) for entities

725f236

fix: change url, dto (request) field, add validation

5b25582

fix: remove service interface

9eb641e

fix: change exception to bad request exception

ac0f515

jcy0308 reviewed Jul 22, 2024

View reviewed changes

shj1081 added 3 commits July 22, 2024 14:34

feat: implement getNoticeList api, minor renaming controller parameter

9fa7335

fix: move @transactional annotation

4608864

fix: remove @DaTa annotation

f448282

refactor: refactor package structure

5d037e0

yesjuhee approved these changes Jul 22, 2024

View reviewed changes

jcy0308 approved these changes Jul 22, 2024

View reviewed changes

2tle approved these changes Jul 22, 2024

View reviewed changes

shj1081 changed the title ~~[FEAT] Notice api 일부 구현 (create, get(단일), update, delete)~~ [FEAT] Notice api 일부 구현 (create, get, update, delete) Jul 22, 2024

shj1081 merged commit f129ccf into feat/9-notice-eventNotice-api/base Jul 22, 2024

shj1081 deleted the feat/9-notice-eventNotice-api/1 branch July 22, 2024 06:40

yesjuhee mentioned this pull request Jul 22, 2024

[FEAT] JobInterview API 구현 #14

Merged

4 tasks

2tle added a commit that referenced this pull request Jul 22, 2024

refactor: apply dto format discussed in PR #12

36e965f

shj1081 mentioned this pull request Jul 23, 2024

[FEAT] Notice API, Notice RestDocs 구현 #18

Merged

4 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[FEAT] Notice api 일부 구현 (create, get, update, delete) #12

[FEAT] Notice api 일부 구현 (create, get, update, delete) #12

shj1081 commented Jul 19, 2024 •

edited

Loading

yesjuhee left a comment

yesjuhee left a comment

yesjuhee commented Jul 20, 2024

jcy0308 commented Jul 21, 2024 •

edited

Loading

shj1081 commented Jul 21, 2024 •

edited

Loading

jcy0308 left a comment

shj1081 commented Jul 22, 2024 •

edited

Loading

shj1081 commented Jul 22, 2024

yesjuhee left a comment

yesjuhee commented Jul 22, 2024

shj1081 commented Jul 22, 2024 •

edited

Loading

2tle left a comment

[FEAT] Notice api 일부 구현 (create, get, update, delete) #12

[FEAT] Notice api 일부 구현 (create, get, update, delete) #12

Conversation

shj1081 commented Jul 19, 2024 • edited Loading

체크 리스트

작업 내역

Notice API 일부 구현 (DTO, Repository, Service, ServiceImpl, Controller)

BaseNoticeEntity, BaseTimeEntity, Notice/EventNotice Entity 수정

비고

작업 branch 2 예정

yesjuhee left a comment

Choose a reason for hiding this comment

yesjuhee left a comment

Choose a reason for hiding this comment

yesjuhee commented Jul 20, 2024

jcy0308 commented Jul 21, 2024 • edited Loading

shj1081 commented Jul 21, 2024 • edited Loading

jcy0308 left a comment

Choose a reason for hiding this comment

shj1081 commented Jul 22, 2024 • edited Loading

shj1081 commented Jul 22, 2024

yesjuhee left a comment

Choose a reason for hiding this comment

yesjuhee commented Jul 22, 2024

shj1081 commented Jul 22, 2024 • edited Loading

2tle left a comment

Choose a reason for hiding this comment

shj1081 commented Jul 19, 2024 •

edited

Loading

jcy0308 commented Jul 21, 2024 •

edited

Loading

shj1081 commented Jul 21, 2024 •

edited

Loading

shj1081 commented Jul 22, 2024 •

edited

Loading

shj1081 commented Jul 22, 2024 •

edited

Loading