Skip to content

Comments

Fix: 위치정보 없는 책갈피 등록 시의 독서노트 불러오기 버그 수정#84

Merged
bee-p merged 6 commits intoreleasefrom
fix/get-bookrecord-with-bookmark
Aug 18, 2025
Merged

Fix: 위치정보 없는 책갈피 등록 시의 독서노트 불러오기 버그 수정#84
bee-p merged 6 commits intoreleasefrom
fix/get-bookrecord-with-bookmark

Conversation

@bee-p
Copy link
Contributor

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

목적🎯

  • 책별 독서노트 조회 API 오류 수정

변경사항🛠️

  • 위치정보 유무 판단 후 response 객체에 정보를 담도록 변경
  • 보안상의 이유로 CI/CD github actions workflow 로직 분리함
    • 앞으로 release 브랜치에 PR을 보내면 -> 테스트만 수행 (PR이 열려있는 상태에서 테스트의 성공 여부 확인 가능)
    • release 브랜치에 merge 및 push 되면 -> 이때 빌드 수행하도록 함

수행한 테스트✏️

  • 위치정보가 없는 책갈피가 등록되어 있는 독서노트를 불러올 경우의 테스트 진행

비고📌

.

bee-p added 3 commits August 18, 2025 14:13
- 위치정보가 없는 책갈피를 추가한 후 독서노트를 불러오면 NullPointerException으로 오류가 났던 상황 확인, 수정함
- 위치정보 없는 책갈피 등록 시 독서노트가 안 불러와지는 현상 수정
- 위치정보 없는 책갈피 등록 후 독서노트 정보 불러오기 상황 테스트
@bee-p bee-p self-assigned this Aug 18, 2025
bee-p added 3 commits August 18, 2025 15:07
- PR은 협업자가 아님에도 외부에서 보낼 수 있고, 이 트리거를 통해 AWS에 악용 파일이 배포될 가능성이 있다고 생각해 CI와 CD를 분리함
@bee-p bee-p merged commit 3b7e94d into release Aug 18, 2025
1 check passed
@bee-p bee-p deleted the fix/get-bookrecord-with-bookmark branch August 18, 2025 06:48
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