Skip to content

Comments

fix: 책갈피 전체 조회 오류 수정#85

Merged
bee-p merged 3 commits intoreleasefrom
fix/get-all-bookmark
Aug 19, 2025
Merged

fix: 책갈피 전체 조회 오류 수정#85
bee-p merged 3 commits intoreleasefrom
fix/get-all-bookmark

Conversation

@bee-p
Copy link
Contributor

@bee-p bee-p commented Aug 19, 2025

목적🎯

  • 책갈피 전체 조회 API 내 오류 수정

변경사항🛠️

  • 책갈피 전체 조회 시 위치 정보의 null 여부를 체크하는 구문을 추가했습니다.
  • 책갈피 전체 조회 response dto 내용 중, location 정보를 List로 보내고 있던 것을 발견해 이를 dto 객체 방식으로 보내도록 수정했습니다.

수행한 테스트✏️

  • 빠른 반영을 위해 Postman으로만 테스트
    • 위치정보가 있을 경우와 없을 경우 둘 다 테스트 했습니다

비고📌

  • 작은 버그 수정이라 서버 버전은 안 바꾸어도 될 것 같아 그대로 두었습니다!

- 책갈피의 위치정보가 없을 경우 null 체크의 부재로 오류 발생, 수정함
- 또한 location 정보가 dto 객체가 아닌 List<String>으로 가고 있던 상황 발견, 인덱싱 접근으로 인한 버그 발생을 우려해 dto 객체로 변경함
@bee-p bee-p self-assigned this Aug 19, 2025
bee-p added 2 commits August 20, 2025 02:29
- 보다 직관적으로 표시되기 위해 job의 이름 변경함
- PR rule을 설정할 때, CI 워크플로우 항목이 자동으로 잡히지 않아 이름 명시함
@bee-p bee-p merged commit e90aa44 into release Aug 19, 2025
1 check passed
@bee-p bee-p deleted the fix/get-all-bookmark branch August 19, 2025 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant