Feature/openapi generator refactoring #6091
Merged
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
jonny-jeahyunchoi
requested review from
sonohoshi,
tyrosine1153 and
eugene-doobu
as code owners
eugene-doobu
reviewed
Oct 8, 2024
| @@ -891,7 +884,7 @@ private static void AppendWebRequestsFromPaths(StringBuilder sb, JsonNode pathsN | |||
|
|
|||
| sb.AppendLine($" public async Task {methodName}({parameterDefinitions}Action<{returnType}> onSuccess, Action<string> onError)"); | |||
| sb.AppendLine(" {"); | |||
| sb.AppendLine($" string url = Url + \"{path}\";"); | |||
| sb.AppendLine($" string url = Url + $\"{path}\";"); | |||
There was a problem hiding this comment.
Suggested change
| sb.AppendLine($" string url = Url + $\"{path}\";"); | |
| sb.AppendLine($" string url = $\"{Url}{path}\";"); |
문자열 보간 쓰는 겸 Url 스트링도 안으로 넣는게 좋을 것 같습니다
There was a problem hiding this comment.
해당 Url은 런타임시 세팅되는 값을 의미하는거라서 코드 생성시점에 정할수가없어서 이렇게 처리했습니다~!
There was a problem hiding this comment.
에디터 타임에서 텍스트가 자동생성 되는 타이밍에 달러 스트링 안으로 Url이 들어가는 형태로 수정되게 할 수는 없을까요?>
There was a problem hiding this comment.
url은 다양한 운영요소로 바뀌게되어서 생성타임에 url을 고정하는것은 안좋은선택인것같아요
단적인 예로 프로토콜은 동일하나, 헤임달url과 메인넷 url이 다를 경우 분기처리가 굉장히 복잡해집니다~!
There was a problem hiding this comment.
아뇨 런타임에 바뀔 수 있게 구성하는건 동일한데, Url자체가 문자열 보간으로 변경될 수 있도록 하면 좋을 것 같아서 했던 말입니다
There was a problem hiding this comment.
확인감사합니다~!
e88947d
에서 수정진행하였습니다~
sonohoshi
approved these changes
Oct 8, 2024
eugene-doobu
approved these changes
Oct 8, 2024
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.
openapi enum 스팩이 c# enum 선언 기준과 다를 경우 예외처리하는 과정 리팩토링진행.
기존: 숫자인경우 앞부분 '' 추가, '-' 를 '' 로 치환. 전처리시 각 예외처리기준에따라 '_' 제거 및 치환
변경: 매핑 테이블을 만들어서 c# enum 선언 기준에 맞춘것으로 강제치환한것과 본래값을 매핑. 및 전처리시 복원하도록 수정.
추가로 관리되는 cs 파일 Path가 변경되어 반영 및 스크립트 정리, 기존 생성된 cs 최신화 진행하였습니다.
메소드 생성규칙을 수정하였습니다.(보다 넓은 범위의 path들을 메소드화시키도록 수정)
기존: path에 api 문자열이 포함된경우.
변경: path에 .이 포함되지 않은 경우.
중복 메소드이름의경우 뒤에 숫자를 추가해서 생성하는식으로 수정하였습니다.(파라메터까지 전부같을경우에만)