Graphql #1451
Closed
Graphql #1451
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
* better code format for java okhttp client * update java petstore samples
* add perl tidy without updating petstore samples * incldue pl file, update samples * use no backup option, perltidy on test.pl
* OpenAPITools#957: add missing type mapping * OpenAPITools#957: generate samples
…PITools#769) * upgrade php-cs-fixer to 2.12 * ran petstore-security-test for php * updating openapi3 php petstore example
* [PHP] Fix deserialize for default responses. Update samples
* [PHP] Update sample files
* Fix the distination ("-o") of petstore-security-test
* Clean up "petstore-security-test" folder
* Update petstore-security-test sample files
* Update petstore OpenAPI3 sample files
* Update README * Remove the codes checking dots contained in path * Update samples
* Drop suport for PHP5.5 on php-client * Update samples bin/php-petstore.sh bin/openapi3/php-petstore.sh * Update security samples bin/security/php-petstore.sh
* add js-beautify support to JS generator * replace tab with space * update samples without JS_BEAUTIFY_PATH
* fix circular import issue in python flask This is very similar change to 3678eaf All it intends to do is fix the problem of cirular imports (which was already fixed for python) in the python flask server. * removal of type hints in quotes
…#656) This fixes an issue that makes the swift 4 api client fail to compile when using enums with a default value. The fix changes the toDefaultValue method to check if the property is of enum type, and then returns a properly formatted value. Looking into the code, I wonder why toEnumDefaultValue is not being used here, which seems to be the case for the java implementation.
* fix[dart]: improved error reporting * rebuilt dart2 petstore * rebuilt dart2 petstore with dartfmt
* Check whether a value is present but null * Update Petstore sample
Remove beta tag Supported nested containers Move repetitive code to inline function Fix docker file Fix CMakeLists.txt when building external project
Support basic query handling Add helpers for primitive de-serialization Remove warnings due to unneeded commas Deserialize basic types in queries Add dependencies chain for external libraries Fixes wrong parameter passed to API
…penAPITools#1008) * Update the .net core sdk to v2.1 and update the asp.net packages used. * Upgrade the SwashBuckle Asp.Net Core package to v3.0.0. * Update the AppSettings json file and add a file used in development. Also, remove the web.config file. * Update the program template to use the web host builder class. * Update the startup class to use the Asp.Net 2.1 paradigms. * Update the launch settings json file. * Update the controller template to derive from the ControllerBase class as aposed to Controller. * Add the SwashBuckle annotations package. * Add the SwashBuckle.AspNetCore.Annotations namespace to the controller template. * Update the Startup template to add comments to the Configuration property and an env parameter to the Configure method. * Update the startup class so we don't need to inject the hosting environment. * Update the program class to have hte recommended CreateWebHostBuilder method from asp.net core 2.1. * Update the asp.net core pet store sample server. * add back aspnetcore 2.0 template via option * remove web.config for aspnet core 2.1
…PITools#761) * Start working on fixing default value in Restbed Server Api Template * fix default value in DefaultCodegen
* Support Gradle 4.10 Gradle 4.10 is bundled with Kotlin 1.60 and Kotlin DSL 1.0-rc1. The new Kotlin DSL isn't binary compatible with the `tasks` registration used in this plugin. Updating to Kotlin DSL 1.0-rc1 with no other changes would require users to update to Gradle 4.10. As a workaround, I've modified the tasks registration being done in OpenApiGeneratorPlugin.kt, so rather than using the Kotlin DSL's invoke, it creates tasks manually against the TasksContainer. This works locally with Gradle 4.7+ for all scenarios in the sample (samples/local-spec). There may be edge cases that I'm unaware of, and we may want to consider defining the minimum supported Gradle version of 4.10 in the next major version of openapi-generator-gradle-plugin if we experience those cases. * Uncomment snapshots repo (commented it during local testing) * update pom.xml for exec gradle plugin
…r" (OpenAPITools#1027) * Revert "[gradle plugin] Support Gradle 4.10 (OpenAPITools#1011)" This reverts commit 131cf94. * Revert "[C++][Restbed] Fix default values for Restbed Server generator (OpenAPITools#761)" This reverts commit f29ba97.
* Add option to set Feign version * add java feign 9x to CI * update usage for Play * fix duplicated artifactId
* Add CLI option to control logging level * Update the property name
* generator for dart jaguar * migrate to openapi-generator * fix locale issue * add auth generation for dart jaguar * fix generation of jaguar api with auth (OpenAPITools#1009) * update deps (OpenAPITools#1016) * add CI tests for dart jaguar * update dart jaguar bin script * trigger build failure * Revert "trigger build failure" This reverts commit a7d8bfd.
* add option to enable scalafmt for code formatting * fix typo
* fix[dart2]: due to stronger type checking in dart 2, maps could not be deserialized * rebuild dart2 petstore
This was referenced Nov 19, 2018
|
Thanks for the PR but your commit (as shown in the Commits tab) is not linked to your Github account, which means this PR won't count as your contribution in https://github.com/OpenAPITools/openapi-generator/graphs/contributors. Let me know if you need help fixing it. |
wing328
reviewed
Nov 22, 2018
...enapi-generator/src/main/java/org/openapitools/codegen/languages/GraphQLServerGenerator.java
Show resolved
Hide resolved
wing328
reviewed
Nov 22, 2018
...enapi-generator/src/main/java/org/openapitools/codegen/languages/AbstractGraphQLCodegen.java
Show resolved
Hide resolved
This was referenced Jan 18, 2022
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.
PR checklist
./bin/to update Petstore sample so that CIs can verify the change. (For instance, only need to run./bin/{LANG}-petstore.shand./bin/security/{LANG}-petstore.shif updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates). Windows batch files can be found in.\bin\windows\.master,3.4.x,4.0.x. Default:master.Description of the PR
cc:
@jfiala
@wing328