Repository navigation
[oneseo] 원서 수정 권한 요청 시간 제한 제거 - #563
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![pr-agent-demo[bot]](https://avatars.githubusercontent.com/in/3949189?s=60&v=4){kind=small}
There was a problem hiding this comment.
-
Overview
- 원서 수정 권한 요청 시 프론트엔드에서 적용하던 9~16시 시간 제한 로직을 제거합니다.
- 관련 안내 문구도 함께 수정합니다.
-
Intent
- 시간대 제한 없이 수정 권한을 요청할 수 있도록 사용자 경험을 개선하려는 목적입니다.
- 기기 시간 기반 하드코딩 조건으로 인한 불필요한 UI 제약을 없애기 위한 변경입니다.
-
Risk
- 전반적으로 낮은 리스크입니다.
- 서버 측 API가 시간 제한을 별도로 검증하지 않는지, 기존 요청/승인 상태 흐름이 영향받지 않는지 확인이 필요합니다.
1 inline comment(s)
| if (!isOneseoWrite) return ['원서 접수 기간이 아닙니다.', 'submit', true]; | ||
|
|
||
| if (!isOneseoModifyRequestTime && editability?.oneseoEditStatus === 'NONE') | ||
| return ['9시부터 16시 이외에는 수정 권한 요청이 제한됩니다', 'submit', true]; |
There was a problem hiding this comment.
🟡 medium/bug
클라이언트에서 시간 제한을 제거해도 서버가 여전히 9~16시를 검증한다면, 버튼은 항상 활성화되지만 시간 외 요청은 서버에서 거부되어 사용자에게 오류 토스트만 표시되는 불일치가 발생합니다. PR 설명에서도 이 부분을 확인하지 못했다고 명시했으니, 병합 전에 서버의 수정 권한 요청 API(POST /oneseo/v3/oneseo/me/request)에서 시간 제한 검증 로직도 함께 제거했는지 확인해 주세요.
This branch was successfully deployed
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.
개요 💡
원서 최종 제출 후 수정 권한을 요청할 때, 프론트에서 9시부터 16시 사이에만 요청할 수 있도록 막고 있었습니다. 이 제한은 서버 응답(
/oneseo/v3/editability)이 아니라GuidePage에서 사용자 기기 시계를 KST로 변환해 판단하는 하드코딩 값이었습니다. 시간과 관계없이 수정 권한을 요청할 수 있도록 해당 제한을 제거하였습니다.작업내용 ⌨️
GuidePage에서isOneseoModifyRequestTime과9시부터 16시 이외에는 수정 권한 요청이 제한됩니다버튼 분기를 제거하였습니다.if (!data) return ['원서 작성하기', ...]보다 먼저 평가되었기 때문에,oneseoEditStatus가NONE이면 원서를 작성하지 않은 사용자도 해당 시간 외에는 버튼이 비활성화될 수 있었습니다. 이번 제거로 이 경우도 함께 해소되었습니다.ClientModals의 수정 권한 요청 모달 안내 문구에서수정 권한 요청은 9시부터 16시 사이까지 가능합니다.문장을 제거하였습니다.NONE/REQUESTED/APPROVED)에 따른 버튼 동작과 원서 접수 기간 검사는 변경하지 않았습니다.리뷰 요청사항 👀
POST /oneseo/v3/oneseo/me/request)에서도 시간대를 검증하는지는 확인하지 않았습니다. 서버가 거부하는 경우 기존onError처리로 서버 메시지가 토스트로 표시됩니다.