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.

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

Apply 코드 수정 #97

Merged
merged 22 commits into from
Apr 9, 2024
Merged

Apply 코드 수정 #97

merged 22 commits into from
Apr 9, 2024

Conversation

sycuuui
Copy link
Contributor

@sycuuui sycuuui commented Apr 3, 2024

#️⃣ 연관된 이슈

ex) #95 #96

📝 작업 내용

  1. 내가 모집 중인 팀에서 팀장포함 총 인원, 팀장 불포함 총 인원, 팀장 불포함 모집된 인원 데이터로 수정
  2. 팀 정보(취소,기간수정,마감)과 지원자 모집결정(지원서 열람,선발,미선발) 요청 시 공고 팀장과 일치 하는지 판단 코드 추가
  3. ApplyService 코드 리펙토링
  4. post가 poroject이면 stackName
    4-1. 공고 지원시 stackName이 null이 가능하도록하여 contest일때 stackName을 안받아도 오류 나지 않도록 진행
    4-2. 공고 정보 요청 시 공고 타입을 확인하여 stack을 null 처리 또는 데이터 전송 (null 처리시 json에 포함되지 않음)

스크린샷 (선택)

💬 리뷰 요구사항(선택)

commit 중 dd2087a 이 부분이 StackName 엔티티와 맵핑해서 진행한 부분인데
"not-null property references a null or transient value : com.gongjakso.server.domain.post.entity.StackName.post" 오류 발생하여 우선 List StackType으로 저장하도록 진행하였습니다!

시간 되시면 이 부분에 대해 보시고 리뷰 부탁드립니다!🙇🏻‍♀️

@sycuuui sycuuui added feat 기능 개발 시 fix 기존 작성된 코드 수정 시 refactor 리팩토링 시 labels Apr 3, 2024
@sycuuui sycuuui requested review from dl-00-e8 and yumzen April 3, 2024 14:02
@sycuuui sycuuui self-assigned this Apr 3, 2024
Copy link
Member

@yumzen yumzen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨어용

Copy link
Contributor

@dl-00-e8 dl-00-e8 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

List 타입으로 한 건 현재 상황에서는 문제없을 것 같습니다!

다만, 추후 Post에서 사용되는 StackName과 Apply에서 사용하는 StackName이 동기화가 안 되면 문제가 발생할 수 있으니까, 구조 변경 작업이 추후 필요할 수 있을 것 같아요.
이건 알고만 계시면 될 것 같아요!!

@sycuuui sycuuui merged commit 5d48e56 into main Apr 9, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 기능 개발 시 fix 기존 작성된 코드 수정 시 refactor 리팩토링 시
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants