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.

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

refactor(snackgame): 게임 검증API의 응답을 통합한다 #181

Merged
merged 2 commits into from
Oct 17, 2024

Conversation

Hwanvely
Copy link
Collaborator

변경 사항

AS-IS

10/14 회의에서 @nijuy 님의 요청이 있었습니다.
기존에는 검증 API의 응답이 두가지 경우로 분리되어 있었습니다.

  1. 일반 스낵을 제거했을 경우 OK만 반환하였습니다.
  2. 황금 스낵을 제거했을 경우 Created와 세션에 대한 정보를 반환하였습니다.

TO-BE

스낵의 종류와 무관하게 Created와 세션에 대한 정보를 담도록 수정하였습니다.

@Hwanvely Hwanvely requested a review from 0chil October 14, 2024 23:02
@Hwanvely Hwanvely self-assigned this Oct 14, 2024
@nijuy
Copy link

nijuy commented Oct 15, 2024

일반 스트릭 = 황금 스낵이 포함되지 않은 스트릭
황금 스트릭 = 황금 스낵이 포함된 스트릭

요청 이유

지금 제가 짜던 코드는 스트릭의 합이 10일 때마다 검증 API를 쏘는 실시간 처리 방식입니다
검증 API의 응답으로 세션 변수값을 업데이트 하고자 통합을 요청 드렸는데요............

잘못 알고 있었음

쪼개서 보내면 기술적으로 순서 보장이 안될 수 있다는 문제를 인지하지 못한 상태였고,
과거 프론트 코드를 살펴보는 과정에서 정확한 파악을 못했습니다 죄송합니다송죄합니

결론

프론트 코드를 기존 방식 (황금 스트릭 제거 시 검증 API로 누적 스트릭 데이터를 보내도록) 수정하고 있어요

그래서 응답 통합은 당장 안 해도 될 거 같아요 잠시 머지 보류 부탁드립니다 ㅠㅠ!!!!!!!!

저 지금 동작구 쪽으로 미안미안댄스 추는 중...................🥺

@0chil
Copy link
Collaborator

0chil commented Oct 15, 2024

빨리 머지 안해서 다행일수가 있다니.. 다행입니다 (?)
해당 작업이 단위가 작지 않은데, 작업 중에 질문/지원요청 언제든 부탁드립니다! @nijuy

@Hwanvely 는 휴업중이시잖아용,, 여긴 맡기고 현생에 집중하셔야 합니다 ㅋㅋㅋ

.body(it)
} ?: ResponseEntity.ok().build()
return ResponseEntity
.status(HttpStatus.CREATED)
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 부분만 OK로 변경해서 머지하겠습니다!

@0chil 0chil merged commit a81ae45 into dev Oct 17, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants