Skip to content

Conversation

@zzaekkii
Copy link
Member

close #173

☑️ 완료 태스크

  • status 파라미터 @NotBlank 제약 삭제
  • status값이 null이라면, 모든 상태의 요청글을 가져오도록 변경
  • 전체 목록 조회 API 응답에 요청글 상태 status 필드도 추가로 전달

🔎 PR 내용

image

원래 PR 리뷰 요청글은 open 혹은 closed 상태로 나뉘어서
전체 목록 조회 시 상태 파라미터 값을 전달하도록 되어있었는데,

@Yeonseo-Jo 연서랑 의논하면서 이와 관련한 변경 사항이 나왔습니다.

  1. openclosed를 모두 보여주는 게 있으면 좋겠다.
  2. 모요이 서비스 기본 홈 화면 PR 리뷰 요청 탭에서는 모두 섞은 걸 보여주자.
  3. 요청글 전체 목록 조회에선 프론트에서 상태값을 필터링하겠다.

1️⃣ status 값 null일 경우, 모든 상태 가져오기

기존에 요청글 목록 조회 시, 쿼리 파라미터로 status값을 open이나 closed 둘 중 하나 넘겨주는 형식이었는데,
파라미터에 status 값이 존재하지 않으면 open + closed를 모두 보여주도록 변경했습니다.

3가지 경우 스크린샷은 하단에 배치해놨습니다.

2️⃣ 모든 상태값을 받았을 때, 상태 구별이 가능하도록 status 추가 전달

image

기존엔 특정 status를 조회 API 호출 시에 쿼리 파라미터로 넘겨주었기에 따로 반환할 필요가 없었으나,
모든 상태의 요청글을 조회할 시 각 요청글의 상태를 구별할 수 있도록 추가로 반환하도록 했습니다.

변경 사항은 PR 리뷰 요청글 전체 조회내 리뷰 요청글 전체 조회에 모두 반영됩니다.

📷 스크린샷

1️⃣ 상태값에 아무 필터링 없을 때

image

2️⃣ OPEN 상태 요청글 전체 조회

image

3️⃣ CLOSED 상태 요청글 전체 조회

image

@zzaekkii zzaekkii self-assigned this Nov 16, 2025
@zzaekkii zzaekkii added :trollface: 재영 Further information is requested 🔨 Refactor 코드 리팩토링 labels Nov 16, 2025
@zzaekkii zzaekkii linked an issue Nov 16, 2025 that may be closed by this pull request
2 tasks
@pull-request-size pull-request-size bot added the size/S Good for newcomers label Nov 16, 2025
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
12.2% Coverage on New Code (required ≥ 80%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@zzaekkii zzaekkii merged commit a79d3a9 into develop Nov 16, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔨 Refactor 코드 리팩토링 size/S Good for newcomers :trollface: 재영 Further information is requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactor: 요청글 기본 상태 변경

2 participants