Repository navigation
fix: 롯데시네마 BookingSeatCount를 잔여 좌석으로 매핑 - #176
Merged
hmmhmmhm merged 2 commits intoAug 18, 2026
Merged
Conversation
공식 GetPlaySequence.BookingSeatCount는 예매된 수가 아니라 예매 가능 좌석 수다. GetSeats 점유 좌석 수와 대조하면 bookedSeats/remainingSeats 라벨이 반대였다. related: NomaDamas/k-skill#598
Contributor
Author
|
안녕하세요. |
This was referenced Aug 17, 2026
Owner
|
확인 감사합니다 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
변경 요약
롯데시네마
GetPlaySequence.BookingSeatCount를bookedSeats로 옮기고remainingSeats = total - BookingSeatCount로 계산하던 매핑이 반대입니다. 공식 필드명은 예매된 수처럼 보이지만, 실제 값은 예매 가능(잔여) 좌석 수입니다.이 PR은 정규화 매핑을 고칩니다.
버그 증명
같은 회차를 공식 LCWS와 대조했습니다. 2026-08-17 롯데시네마 안양일번가(
CinemaID=3032), 19:50 오디세이 5관:GetPlaySequence.TotalSeatCountGetPlaySequence.BookingSeatCountbookedSeats/remainingSeatsGetSeats.BookingSeats.length(실제 점유)GetSeatsSeatStatusCode=0(실제 빈 좌석)BookingSeatCount는 잔여 159석과 같고,GetSeats점유 34석과 같습니다. 기존 라벨대로 읽으면 잔여가 많은 회차가 매진처럼 보입니다.네트워크 분석 문서에 있던
GetSeats.BookingCount=4vsBookingSeatCount=28불일치도 같은 이유입니다.32 - 28 = 4로 맞습니다.관련 이슈
체크리스트
npm run typecheck통과npm run lint통과npm run lint:biome통과npm test통과 (1644)테스트 방법
픽스처
TotalSeatCount=32,BookingSeatCount=28에서remainingSeats는 28,bookedSeats는 4여야 합니다.참고 사항
공개
daiso@1.0.10CLI//api/lottecinema/seats응답을 깨는 breaking change입니다. 라벨을 그대로 믿던 클라이언트는 잔여/예매가 뒤집혀 있었습니다.