20250202/next result type 오송민 기술블로그 제출 #119
Open
Conversation
This file contains 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
obvoso
requested review from
day1companyhaewooklee and
day1chan99k
as code owners
hjh010501
reviewed
Feb 8, 2025
|
|
||
| `Result<T>`를 사용하는 비동기 요청을 처리하는 함수를 정의할 차례입니다. | ||
|
|
||
| ## 기존 에러 처리 방식 |
There was a problem hiding this comment.
이거를 Result 타입 설명 전으로 올리는 것이 저는 조금 더 자연스럽다는 생각이 들기는 하는 것 같습니다..! Result 타입 -> 기존에 에러 처리로 갑자기 이동하다보니 읽었을 때 조금 어려웠던 것 같습니다
There was a problem hiding this comment.
기존 에러 처리 -> Result type 설명 순으로 변경 했습니다! 감사합니다!
|
|
||
| ### 클라이언트 요청일 경우 | ||
|
|
||
| 클라이언트 요청에 대한 에러 메세지는 `ky`의 `beforeRequest`에서 처리해주고 있었습니다. |
There was a problem hiding this comment.
비동기 요청에 대한 에러 처리인데 HTTP 요청에 대해서만 예시가 있어서.. 요거 외에도 예시를 몇개 더 만들어 주셔도 좋을 것 같습니다!
There was a problem hiding this comment.
비동기 요청에 대한 에러 처리인데 HTTP 요청에 대해서만 예시가 있어서.. 요거 외에도 예시를 몇개 더 만들어 주셔도 좋을 것 같습니다!
말씀해주신대로 제목을.. 변경하는 쪽으로 수정 했습니다.. 감사합니다!
|
|
||
| ### 흐름 도식화 | ||
|
|
||
| 요청의 흐름을 도식화 하면 아래와 같습니다. |
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.
No description provided.