프로젝트 게시글 저장 api 구현 #59

Sehee-Lee-01 · 2024-02-20T08:22:34Z

🎫 관련 이슈

Resolves #31

✅ 구현 내용

프로젝트 게시글 저장 api 구현

💬 코멘트

아래 사항은 나중에 나눠서 올리겠습니다!
- 유효성 검사
- 테스트 코드
- swagger
🚨 프로젝트 상세 조회 기능 구현 #56 => 머지한 후에 머지해주세요!!!!!!!!!!!!

생성

…-create-project

github-actions · 2024-02-20T08:35:44Z

📝 Jacoco Test Coverage

Overall Project	45.95% `-33.53%`	❌
Files changed	7.5%	❌

File	Coverage
FileType.java	100%	🍏
UserSearchResponse.java	100%	🍏
UserSummary.java	72.73% `-27.27%`	❌
MemberSaveRequest.java	0%	❌
ProjectSkillSaveRequest.java	0%	❌
ProjectSaveRequest.java	0%	❌
ProjectService.java	0%	❌
FileService.java	0%	❌
ProjectSkillService.java	0%	❌
MemberService.java	0%	❌
ProjectResponse.java	0%	❌
MemberSummary.java	0%	❌
OverviewImageSummary.java	0%	❌
ProjectSkillSummary.java	0%	❌
File.java	0% `-33.33%`	❌
Project.java	0% `-32.76%`	❌
ProjectSkill.java	0% `-44.44%`	❌
ProjectController.java	0%	❌
Member.java	0% `-38.1%`	❌

# Conflicts: # src/main/java/sixgaezzang/sidepeek/projects/dto/response/MemberSummary.java # src/main/java/sixgaezzang/sidepeek/projects/service/ProjectService.java

uijin-j

프로젝트 저장이 저희의 메인 기능이라 정말 복잡하네요ㅠㅠ 넘넘 고생하셨습니다🥹🥹
기술스택, 프로젝트 멤버 저장까지.. 짱이에요..👍🏻👍🏻

아마 이것저것 생각할 부분이 많아서 요청/응답 DTO와 도메인에 유효성 검사 로직을 아직 작성하지 않으신 것 같은데..! 천천히 추가해 주시면 감사할 것 같습니다🥰
(헉 죄송해요..! 유효성 검사는 다음 PR에 올리신다는 글을 리뷰 다하고 봤네요..!!!! 전 똥멍청이 입니다😇😭 해당부분 리뷰는 무시해주세융..)

src/main/java/sixgaezzang/sidepeek/projects/domain/Project.java

uijin-j · 2024-02-21T08:08:25Z

src/main/java/sixgaezzang/sidepeek/projects/dto/request/ProjectSaveRequest.java

+    String name,
+    String subName,
+    String overview,
+    String thumbnailUrl,
+    String githubUrl,
+    String deployUrl,
+    @JsonFormat(pattern = "yyyy-MM-dd HH:mm:ss")
+    LocalDateTime startDate,
+    @JsonFormat(pattern = "yyyy-MM-dd HH:mm:ss")
+    LocalDateTime endDate,
+    Long ownerId,
+    String description,
+    String troubleShooting,
+
+    List<String> overviewImageUrls,
+    List<ProjectSkillSaveRequest> techStacks,
+    List<MemberSaveRequest> members


Spring Validation 추가해도 좋을 것 같습니당!
✔️ 필수값: 제목, 개요, 기술 스택, 깃허브 링크, 썸네일
✔️ 길이제한: 제목, 소제목 개요

ㅠㅠㅠㅠ감사합니다!! 다음 피알에 반영하겠습니당!♥♥

src/main/java/sixgaezzang/sidepeek/projects/dto/response/MemberSummary.java

src/main/resources/db/migration/V1__init.sql

src/main/java/sixgaezzang/sidepeek/projects/domain/file/FileType.java

src/main/java/sixgaezzang/sidepeek/projects/domain/member/Member.java

uijin-j · 2024-02-21T08:23:20Z

src/main/java/sixgaezzang/sidepeek/projects/domain/member/Member.java

+                  String role, String nickname) {
        this.user = user;
+        this.userId = userId;
        this.project = project;
-        this.authority = authority;
+        this.projectId = projectId;
        this.role = role;
        this.nickname = nickname;


userId또는 user가 null이면 비회원 멤버이기 때문에 nickname이 필수인데, 유효성 검사 로직을 나중에 추가해 주셔도 될 것 같아용! (+ project, role도 필수값이라 나중에 null인지 확인하는 유효성 검사가 있으면 좋을 것 같습니당!)

src/main/java/sixgaezzang/sidepeek/projects/domain/file/File.java

yenzip

역시 메인 도메인이기에 코드 양이 상당하네요..!
혹시 conflict 관련해서 도움이 필요하다면 언제든 삐삐쳐주세요! 🤙

정말 고생 많으셨습니다 ❤️‍🔥

.github/workflows/ci.yml

src/main/java/sixgaezzang/sidepeek/projects/controller/ProjectController.java

src/main/java/sixgaezzang/sidepeek/projects/domain/file/File.java

src/main/java/sixgaezzang/sidepeek/projects/domain/member/Member.java

src/main/resources/db/migration/V1__init.sql

github-actions · 2024-02-22T01:12:14Z

📝 Jacoco Test Coverage

Overall Project	45.48% `-16.02%`	❌
Files changed	8.76%	❌

File	Coverage
UserSearchResponse.java	100%	🍏
UserSummary.java	72.73% `-9.09%`	🍏
MemberSaveRequest.java	0%	❌
ProjectSkillSaveRequest.java	0%	❌
ProjectSaveRequest.java	0%	❌
ProjectService.java	0% `-36.71%`	❌
FileService.java	0%	❌
ProjectSkillService.java	0%	❌
MemberService.java	0%	❌
MemberSummary.java	0% `-23.08%`	❌
Project.java	0% `-5.17%`	❌
ProjectSkill.java	0% `-33.33%`	❌
ProjectController.java	0% `-72.41%`	❌
Member.java	0% `-27.78%`	❌

github-actions · 2024-02-22T01:50:22Z

📝 Jacoco Test Coverage

Overall Project	45.48% `-16.02%`	❌
Files changed	8.76%	❌

File	Coverage
UserSearchResponse.java	100%	🍏
UserSummary.java	72.73% `-9.09%`	🍏
MemberSaveRequest.java	0%	❌
ProjectSkillSaveRequest.java	0%	❌
ProjectSaveRequest.java	0%	❌
ProjectService.java	0% `-36.71%`	❌
FileService.java	0%	❌
ProjectSkillService.java	0%	❌
MemberService.java	0%	❌
MemberSummary.java	0% `-23.08%`	❌
Project.java	0% `-5.17%`	❌
ProjectSkill.java	0% `-33.33%`	❌
ProjectController.java	0% `-72.41%`	❌
Member.java	0% `-27.78%`	❌

github-actions · 2024-02-22T05:15:25Z

📝 Jacoco Test Coverage

Overall Project	44.6% `-17.53%`	❌
Files changed	7.92%	❌

File	Coverage
UserSearchResponse.java	100%	🍏
UserSummary.java	72.73% `-9.09%`	🍏
MemberSaveRequest.java	0%	❌
ProjectSkillSaveRequest.java	0%	❌
ProjectSaveRequest.java	0%	❌
ProjectService.java	0% `-36.71%`	❌
FileService.java	0%	❌
ProjectSkillService.java	0%	❌
MemberService.java	0%	❌
MemberSummary.java	0% `-23.08%`	❌
Project.java	0% `-5.17%`	❌
ProjectController.java	0% `-72.41%`	❌
Member.java	0% `-13.33%`	❌

github-actions · 2024-02-22T08:28:07Z

📝 Jacoco Test Coverage

Overall Project	45.28% `-14.95%`	❌
Files changed	8.52%	❌

File	Coverage
AuthService.java	100%	🍏
UserSearchResponse.java	100%	🍏
LoginResponse.java	100%	🍏
UserSummary.java	72.73% `-9.09%`	🍏
MemberSaveRequest.java	0%	❌
ProjectSkillSaveRequest.java	0%	❌
ProjectSaveRequest.java	0%	❌
ProjectService.java	0% `-36.71%`	❌
FileService.java	0%	❌
ProjectSkillService.java	0%	❌
MemberService.java	0%	❌
MemberSummary.java	0% `-23.08%`	❌
Project.java	0% `-5.17%`	❌
ProjectController.java	0% `-72.41%`	❌
Member.java	0% `-13.33%`	❌

Sehee-Lee-01 added 26 commits February 18, 2024 20:51

merge: 프로젝트 생성 브랜치에 다른 브랜치 작업 업데이트

bb7e888

chore: S3 설정 마무리

77c5474

Merge branch 'feat/#28-upload-files' into feat/#4-create-project

a5c45fe

refactor: fileType 재정의

d0cd47d

feat: 프로젝트 서비스 관련 레포지토리 생성

21b1145

feat: 프로젝트 기능 관련 Service

d34703d

생성

rename: dto 이름 변경

d24154d

Merge remote-tracking branch 'origin/feat/#5-read-project' into feat/#4…

900b28e

…-create-project

rename: dto 이름 변경

fd9189c

refactor: 불필요한 레포지토리 삭제

aabbaa1

feat: 프로젝트 팀원 저장 서비스 메서드 구현

2708d58

feat: 프로젝트 스킬 저장 서비스 메서드 구현

ddc2549

fix: 프로젝트 id 설정 추가

2966f95

feat: 프로젝트 관련 서비스 Transactional 설정

8956e8b

refactor: 생성자에 ownerId 적용

eabb413

feat: Project 객체로 바꾸는 toEntity 메서드 구현

b5d1b43

feat: 프로젝트 저장 서비스 기능 구현

2ac4c2f

feat: 프로젝트 저장 컨트롤러 기능 구현

d4bba0c

refactor: 불필요한 파일 삭제

42f71d0

fix: bean 이름이 겹쳐서 s3 파일 저장 관련 도메인 이름 수정

b72a5a4

refactor: 필드에 private final 적용

882009e

refactor: OverviewImageSaveRequest 삭제 후 String 타입으로 변경

037435d

fix: 잘못된 @RequestBody import 수정

f755965

refactor: 필드에 복수형 설정(overviewImageUrls)

ad6cca3

merge: dev 브랜치 업데이트

f8ac6c6

Merge branch 'feat/#28-upload-files' into feat/#4-create-project

9950cd3

Sehee-Lee-01 added 🚀 더 만들어봤슈 기능 구현 🌳 밥 좀 드셔유 천천히 리뷰 labels Feb 20, 2024

Sehee-Lee-01 added this to the 🚀 2차 스프린트 milestone Feb 20, 2024

Sehee-Lee-01 self-assigned this Feb 20, 2024

Sehee-Lee-01 requested review from yenzip and uijin-j as code owners February 20, 2024 08:22

Sehee-Lee-01 changed the title ~~Feat/#04 create project~~ 프로젝트 게시글 저장 api 구현 Feb 20, 2024

Sehee-Lee-01 added 2 commits February 20, 2024 17:24

style: 체크스타일 오류 수정

4cc6b55

chore: [email protected]로 버전 업

f639255

Merge branch 'dev' into feat/#4-create-project

0426633

This was linked to issues Feb 20, 2024

프로젝트 업로드 기능 구현 #4

Closed

프로젝트 게시글 저장 api 구현 #31

Closed

Merge branch 'dev' into feat/#4-create-project

3f29d3d

# Conflicts: # src/main/java/sixgaezzang/sidepeek/projects/dto/response/MemberSummary.java # src/main/java/sixgaezzang/sidepeek/projects/service/ProjectService.java

uijin-j reviewed Feb 21, 2024

View reviewed changes

yenzip approved these changes Feb 21, 2024

View reviewed changes

Sehee-Lee-01 added 2 commits February 22, 2024 09:52

refactor: 프로젝트관련 도메인에서 프로젝트 id가 아닌 프로젝트 객체를 다루도록 수정

a281929

fix: 조회수 안올라가는 것 수정

e160f75

fix: github_url 필수로 설정

8affa13

Sehee-Lee-01 added 6 commits February 22, 2024 11:03

fix: 변수명 role로 변경

19f3238

fix: yml 파일명 변경

9c10d57

style: 프로젝트 FileService 코드 스타일 수정

0656b48

feat: 요청으로 들어온 회원 멤버가 존재하지 않는 회원인 경우 확인하는 로직 추가

47b16cb

refactor: id 필드 삭제

94d66aa

refactor: service 레이어 가독성 고려하여 리팩토링

58520de

Sehee-Lee-01 added 2 commits February 22, 2024 16:47

Merge branch 'dev' into feat/#4-create-project

e6b1400

fix: UserSummaryResponse 이름 변경

e523e96

Sehee-Lee-01 merged commit 3d886dd into dev Feb 22, 2024
2 checks passed

Sehee-Lee-01 deleted the feat/#04-create-project branch February 22, 2024 08:28

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

프로젝트 게시글 저장 api 구현 #59

프로젝트 게시글 저장 api 구현 #59

Sehee-Lee-01 commented Feb 20, 2024 •

edited

Loading

github-actions bot commented Feb 20, 2024

uijin-j left a comment •

edited

Loading

uijin-j Feb 21, 2024 •

edited

Loading

Sehee-Lee-01 Feb 21, 2024

uijin-j Feb 21, 2024 •

edited

Loading

yenzip left a comment •

edited

Loading

github-actions bot commented Feb 22, 2024

github-actions bot commented Feb 22, 2024

github-actions bot commented Feb 22, 2024

github-actions bot commented Feb 22, 2024

프로젝트 게시글 저장 api 구현 #59

프로젝트 게시글 저장 api 구현 #59

Conversation

Sehee-Lee-01 commented Feb 20, 2024 • edited Loading

🎫 관련 이슈

✅ 구현 내용

💬 코멘트

github-actions bot commented Feb 20, 2024

📝 Jacoco Test Coverage

uijin-j left a comment • edited Loading

Choose a reason for hiding this comment

uijin-j Feb 21, 2024 • edited Loading

Choose a reason for hiding this comment

Sehee-Lee-01 Feb 21, 2024

Choose a reason for hiding this comment

uijin-j Feb 21, 2024 • edited Loading

Choose a reason for hiding this comment

yenzip left a comment • edited Loading

Choose a reason for hiding this comment

github-actions bot commented Feb 22, 2024

📝 Jacoco Test Coverage

github-actions bot commented Feb 22, 2024

📝 Jacoco Test Coverage

github-actions bot commented Feb 22, 2024

📝 Jacoco Test Coverage

github-actions bot commented Feb 22, 2024

📝 Jacoco Test Coverage

Sehee-Lee-01 commented Feb 20, 2024 •

edited

Loading

uijin-j left a comment •

edited

Loading

uijin-j Feb 21, 2024 •

edited

Loading

uijin-j Feb 21, 2024 •

edited

Loading

yenzip left a comment •

edited

Loading