Skip to content

Conversation

@Jeongminyooa
Copy link
Collaborator

@Jeongminyooa Jeongminyooa commented Jun 25, 2025

#96

📝작업 내용

13번째 선물(MAX_SIZE 초과 케이스) 못 받게 수정

"13번째 기록 완료 후 홈 화면에 선물받기 툴팁 미노출 및 선물받기 안되어야 함."

  • 해당 이슈 해결하기 위하여, 첫번째 커밋 메시지처럼 isOpened를 true로 히든 아이템이 받아지게 세팅하려고 했으나, 생각해보니 우리 로직해서는 해당 컬럼으로 히든 아이템 처음 받고 읽었을 때의 툴팁 출력 여부를 구별하고 있었음.
  • 따라서 종식 오빠 요청대로, isNotOpened의 flag 값에 대하여 선물 12개 받았을 때는 false로 내려주게 해서 선물 열기에 대한 API 자체를 보내지 않도록 구성

이미 히든 아이템 열었는데 hidden read한 경우 예외 처리

  • 히든 아이템을 이미 열었는데 또 열려고 하면 쿼리에서 500 에러가 떨어짐.
  • 해당 에러 대응 및 앞으로의 모든 예외 케이스 동일을 위해 Exception Handler 코드 도입
  • 예외 메시지 및 코드를 커스텀하고 적절하게 핸들링하기 위하여 전역적 핸들러 도입
  • 타 API에 대해서도 동일 응답 구조로 형성하기 위해 리팩토링 필요함. @k906506

@Jeongminyooa Jeongminyooa self-assigned this Jun 25, 2025
@Jeongminyooa Jeongminyooa added the bug Something isn't working label Jun 25, 2025
@github-actions
Copy link

Unit Test Results

1 tests   1 ✔️  0s ⏱️
1 suites  0 💤
1 files    0

Results for commit ebde2c7.

Copy link
Collaborator

@k906506 k906506 left a comment

Choose a reason for hiding this comment

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

고생했어~ 다른 API도 Exception 정의해서 수정해볼게

@Jeongminyooa Jeongminyooa merged commit a06f398 into dev Jun 28, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants