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.

Sign up for GitHub

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

Jump to bottom

[FIX] Api 오류들 수정 #131

Merged
merged 4 commits into from
Feb 21, 2024
Merged

[FIX] Api 오류들 수정 #131

merged 4 commits into from
Feb 21, 2024

Conversation

happyjamy
Copy link
Collaborator

@happyjamy happyjamy commented Feb 21, 2024 •
edited
Loading

🚀 개발 사항

스웨거 문서 페이지네이션 정보 누락 오류 수정

  • 페이지네이션 정보가 누락 되었습니다. 쿼리 파라미터로 받는 정보들은 메서드 위에 @parameters 로 선언 해줘야만 스웨거 문서 상에서 볼 수 있습니다.
  • 제네릭을 사용해서 return value 를 명시하는 컨트롤러는 content 를 따로 지정하지 않아야 제대로 return 값이 나타납니다. 현재 경우도 동일한 경우로 content 지정한 부분을 제거하였습니다.

프로젝트 요청 보내기 유효성 검사 쿼리 오류 수정

  • 본인이 보낸 대기상태의 요청이 있는 경우 다른 프로젝트에 요청을 보낼 수 없습니다. 해당 유효성 검사에서 'PENDING' 상태의 요청 존재 여부만 체크해야 하는데 모든 상태의 요청 존재 여부를 체크해 오류가 발생 하였습니다. 따라서 쿼리에 'PENDING' 상태의 요청만 가져오게 수정 하였습니다.

이슈 번호

  • close

특이 사항 🫶

@happyjamy happyjamy added 🐞 Bug 버그 사항 🎯 MVP1 mvp1 개발사항 labels Feb 21, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 21, 2024 •
edited
Loading

Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit cc1d49f.

♻️ This comment has been updated with latest results.

@happyjamy happyjamy changed the title [FIX] 스웨거 문서 페이지네이션 정보 누락 오류 수정 [FIX] Api 오류들 수정 Feb 21, 2024
JIN-076 and others added 2 commits February 22, 2024 01:38
@JIN-076
[REFACT] 프로젝트 생성일 유효성 검사, 프로젝트 상태 유효성 검사, 프로젝트 상태 필드 추가 (#132)
17f9ad0 
* [REFACT] 프로젝트 카드 생성일에 대한 유효성 검사 로직 구현

* [REFACT] 대기중에만 요청이 생성되도록 프로젝트 카드 상태에 대한 유효성 검사 추가

* [REFACT] 대기중에만 요청이 생성되도록 프로젝트 카드 상태에 대한 유효성 검사 추가

* [REFACT] 프로젝트 상태 추가

* [REFACT] queryDsl 로직 수정, 프로젝트 상태 매핑 추가
@happyjamy
Merge branch 'main' into develop
cc1d49f
@happyjamy happyjamy self-assigned this Feb 21, 2024
@happyjamy happyjamy merged commit 95f7860 into main Feb 21, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JIN-076 JIN-076 JIN-076 approved these changes

@tidavid1 tidavid1 Awaiting requested review from tidavid1

Assignees

@JIN-076 JIN-076

@happyjamy happyjamy

Labels
🐞 Bug 버그 사항 🎯 MVP1 mvp1 개발사항
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@happyjamy @JIN-076