feat: LLM API 키 콘솔 화면 - #31
Merged
Merged
Conversation
모든 한도가 선택 입력이라 비워 두는 것이 정상 경로다. 화면이 그렇게 말하도록 안내와 요약 문구를 맞췄고, 초안 호환성 검사는 이 종류가 쓰지 않는 값(숫자, null, 모르는 키)을 전부 걸러 다른 종류의 초안이 빈 화면으로 되살아나지 않게 한다.
부여 한도 칸은 비운 채로 연다. 희망값을 프리필하면 승인 한 번이 곧 희망대로 부여가 되어 검토가 사라지므로, 희망값은 각 칸 아래에 참고로만 적는다. 동시 요청 수는 신청에 대응하는 항목이 없어 승인자만 정한다는 사실도 화면에서 말한다.
내 신청 목록과 신청 상세가 vm 필드를 직접 읽어, VM 아닌 신청은 신청자에게 빈 칸으로 보였다. 두 화면을 종류 모듈로 넘기면서 읽기 항목을 관리자 화면과 한 벌로 쓴다. 신청의 사실은 읽는 사람의 것이 아니고, 표를 두 벌 두면 한쪽만 낡는다. 신청자 항목은 두 상세 화면의 머리말이 이미 밝히므로 카드에서 뺐다.
서버는 신청과 승인 본문의 하위 객체 경로로 필드 오류를 싣는다(vm.imageId, vm.grantedSlug). 콘솔은 평평한 이름으로 받고 있어 VM 신청의 422가 어느 칸에도 붙지 못했다. 최상위 필드(workspaceId, purpose, grantedEndDate, type)는 접두사가 없으므로 그대로 두고, vm 아래 항목만 서버가 쓰는 경로로 맞췄다.
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.
📝 작업내용
⭐️ 검증
💬 리뷰 포인트