-
Notifications
You must be signed in to change notification settings - Fork 27
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
개발자센터 api 문서 예시 요청/응답 값 보여주기 #411
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@disjukr |
request body를 입력하기 위한 공간이 많이 부족한 것 같은데요, 이 부분이 불편하지 않을까요? |
네넹 제가 보기에도 UI 컴포넌트 4개를 구겨넣기보다는 탭 형태로 TRY랑 SAMPLE을 나눠 둔다던가 하는 게 나을 것 같아요 |
Request 값에 따라서 Request Sample이 생성되는 방식이라 둘을 분리하기가 뭔가 애매하네요... 혹시 좋은 방법이 없을까요? |
음 아니면 Request/Response를 탭 UI로 만들어도 좋을 것 같네요 (요청 실행 시 자동으로 Response 탭으로 넘어가도록 하고) |
제 생각에는 Response Sample을 한눈에 보는것도 중요할 것 같아서 Response 결과만 탭으로 넘기고, 나머지 3개의 그리드 크기를 조정해서 Request에 비중을 더 주면 괜찮을 것 같은데 어떤가요? |
음 한번 적용해보고 크기를 봐야지 감을 잡을 수 있을 것 같은데, 개인적으로 생각하기에는 높이가 너무 작아지면 여러모로 너무 불편해질 것 같아서 (특히 응답 예제는 상당히 길어질 수도 있을 것 같아가지고) 좀 꺼려지긴 하네요 |
네 그러면 탭을 한번 적용해보고 생각해봐야겠네요 |
{snippetSignal.value ? ( | ||
<MonacoEditor | ||
key={snippetSignal.value} | ||
init={(monaco, domElement) => | ||
monaco.editor.create(domElement, { | ||
...commonEditorConfig, | ||
value: snippetSignal.value || "", | ||
language: targetInfoSignal.value?.key || "plaintext", | ||
readOnly: true, | ||
}) | ||
} | ||
/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
사실 Monaco readonly로 넣는 게 나을지 그냥 Shiki 같은 거 돌리는 게 나을지 잘 모르겠긴 하네요
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
일단 통일성 있게 같은 에디터로 작업했는데, 어차피 readonly면 그렇게 하고 복사 버튼 하나 넣어도 상관없겠군요
2f47545
to
69c145c
Compare
69c145c
to
7f0e6a8
Compare
d32890d
to
7033507
Compare
@XiNiHa 일단 Request / Response 탭을 만들어서 분리해봤어요 |
Co-authored-by: Cosmo Shin (신의하) <[email protected]>
Co-authored-by: Cosmo Shin (신의하) <[email protected]>
Co-authored-by: Cosmo Shin (신의하) <[email protected]>
Co-authored-by: Cosmo Shin (신의하) <[email protected]>
Co-authored-by: Cosmo Shin (신의하) <[email protected]>
Co-authored-by: Cosmo Shin (신의하) <[email protected]>
05dc57a
to
80dd0a7
Compare
Co-authored-by: Cosmo Shin (신의하) <[email protected]>
작업 내용
openapi.json
스키마에 정의되어 있는example
프로퍼티로 예시 응답을 보여주는ResExample
컴포넌트를 구현했습니다.example
프로퍼티 값으로 설정되도록 했고,string
타입의 기본 값을 빈 문자열에서 파라미터 이름으로 변경했습니다.Parameter["type"]
이falsy
일 경우에Parameter["schema"]["type"]
값으로 타입이 결정되지 않고object
로 잘못 설정되는 버그를 수정했습니다.ReqSample
컴포넌트를 구현했습니다.설명
string
타입의 기본값을 파라미터 이름으로 설정한 이유는 요청 샘플에서string
타입의 Path 파라미터를 처리하기가 곤란했기 때문입니다. 가령, /payments/{paymentId}/cash-receipt 엔드포인트의 경우에는paymentId
파라미터가 빈 문자열이면/payments//cash-receipt
형태로 샘플 코드가 생성되기 때문에 이를 방지하기 위해 기본값을 파라미터 이름으로 설정했습니다.