[Hotfix] 어드민 QA 반영 - #299
Merged
Merged
Conversation
서버 데이터 개수만큼만 setValue를 호출해 그 이상 인덱스를 비워주지 않았음. 배포 직후 stale한 캐시로 폼을 다시 채우는 타이밍과 겹치면 방금 삭제한 질문이 화면에 부활하는 현상으로 이어져, 모든 인덱스를 순회하며 서버에 없는 값은 명시적으로 빈 문자열로 덮어쓰도록 수정.
파트별 질문 칸 수를 항상 최소 3개로 고정해서 실제 저장된 개수보다 많은 빈 칸이 표시되던 문제를 제거하고, 신규 파트는 최소 1개만 기본 노출되도록 변경. 화면에 보이는 칸(질문/답변)은 모두 필수로 간주해 하나라도 비어있으면 배포 시 에러를 띄우고 막도록 검증 로직을 추가. 화면에 보이는 칸 수를 폼(recruitQuestionCount)에 반영해 배포 유효성 검사에서 참조하도록 함.
지원서/공홈이 전체 일정의 첫 번째 항목을 이름으로 식별하는데, 관리자가 직접 입력한 세션명이 저장되면서 식별이 깨지는 문제가 있었음. FIRST_SCHEDULE_SESSION_NAME 상수를 도입해 0번째 행은 입력 UI를 비활성화하고, 서버 전송 시와 서버 데이터 동기화 시 모두 강제로 'OT'로 고정하도록 수정. 과거에 순서가 뒤섞여 저장된 데이터에 대비해 이름 기준으로 재정렬하는 폴백도 추가.
'최신 소식은 배포 버튼을 거치지 않고 즉시 배포가 돼요' 설명이 삭제 확인 맥락과 맞지 않아 혼란을 줘서 제거.
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
2 participants
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.
✨ 구현 기능 명세