Feat(csharp-dotnet-framework): Template updates#2
Open
saigiridhar21 wants to merge 3 commits into
Open
Conversation
saigiridhar21
changed the title
saigiridhar21
pushed a commit
that referenced
this pull request
Aug 9, 2020
OpenAPITools#5750) * Added support Retrofit2(RxJava/RxJava2/Coroutines) to Kotlin client code generator * Added generated samples for Retrofit2(RxJava/RxJava2/Coroutines) on Kotlin * Fixed generating retrofit2 without Rx/Coroutines * Fixed MultipartBody template, remove redundant space after MultipartBody.Part annotation * Fix documentation diff. * Fix generating build.gradle file for samples. Add dependencies for rxJava/rxJava2 and rx retrofit's adapter. * Update generated sample with rx dependencies * Update generated sample with coroutines * Update generated sample with RxJava 2 dependencies * Update and refactoring scripts for sample generation * Update generated sample code * revert changes by mistake * revert changes by mistake #2 * Fix return type for first RxJava * Add RxJavaCallAdapterFactory to ApiClient scheme for RxJava 1/2 * Fix script loggin * Update generated code for RxJava 1/2 * Fix kotlin.md documentation by script export_docs_generators.sh * Update Kotlin samples project in pom.xml * Revert "Update Kotlin samples project in pom.xml" This reverts commit 9de4d0b * Fixed "" wrapping number types for annotation value Fixed generating polymorphic interfaces for Gson, interface fields can't be marked with @SerializedName annotation, it's enough to mark it in child model. Fixed instantiationTypes(array, list, map) for Kotlin generator * Update Kotlin samples project after last fixes * Update Kotlin samples project for kotlin-jvm-* * Update Kotlin readme.md documentations * Update Kotlin client sample changes * Fixed encoding braces for generics model * Update Kotlin client sample changes * Update Kotlin client sample after merge with master * Fixed adding empty braces for inherit from Map/Array * Update sample model after last fix with empty braces * Revert adding @SerializedName to Kotlin interface fields through @get:SerializedName as for Jackson * Update Kotlin client samples with adding @get:SerializedName
saigiridhar21
pushed a commit
that referenced
this pull request
Aug 9, 2020
…Body, Set List as default collection for Kotlin data class (OpenAPITools#6456) * Replace typeMapping kotlin.Array with kotlin.collections.List, because its doesn't work with Kotlin Data Classes(required manually implementing hashCode&equals) Replace typeMapping kotlin.Array<kotlin.Byte> with efficient primitive implementation kotlin.ByteArray Replace instantiationTypes array to kotlin.collections.ArrayList, for extending Array we should provide element count to its constructor. * Fixed path(removed slash on begin path) for Jvm Kotlin Retrofit * Fixed handling optional bodyParams for Jvm Kotlin Retrofit * Fix kotlin tests * Fixed code format for detekt inspections #2 * revert formatting * Added ability to generate optional @query params. * Update Kotlin docs. * Update Kotlin openapi3 client samples * Update Kotlin client samples * Update Kotlin openapi3 client samples after merge with master * Revert hardcoding List instead Array * Set List as default collectionType * Update Kotlin samples * Fixed Kotlin multiplatform api template. fix mapper for Array type. Fixed Kotlin multiplatform api template Code style. * Update Kotlin multiplatform sample * Fix Kotlin multiplatform template code style * Update Kotlin multiplatform sample * Fix Kotlin multiplatform converting Array to List in Api. * Update Kotlin multiplatform sample #3 * Fix Kotlin tests * Fix Kotlin jackson Application.kt * Fix Kotlin tests #2 * Fix merge conflict with master * Generate samples after merge with master * Generate samples after merge with master #2 * Generate samples after merge with master #3 * Generate samples after merge with master OpenAPITools#4 * update kotlin samples * update all samples Co-authored-by: William Cheng <wing328hk@gmail.com>
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
2 participants
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.
Changes that are made: