Skip to content
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

Graphql client (Apollo Server Generator) #1444

Closed

Conversation

renepardon
Copy link
Contributor

PR checklist

  • Read the contribution guidelines.
  • Ran the shell script under ./bin/ to update Petstore sample so that CIs can verify the change. (For instance, only need to run ./bin/{LANG}-petstore.sh and ./bin/security/{LANG}-petstore.sh if 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\.
  • Filed the PR against the correct branch: master, 3.4.x, 4.0.x. Default: master.
  • Copied the technical committee to review the pull request if your PR is targeting a particular programming language.

Description of the PR

I've refactored the draft of GraphQLSchemaGenerator and implemented a GraphQLServerGenerator as well. I think it's not yet perfect but a somewhat working server can be generated from the Petstore example documentation. I filled the pull request against the graphql_client branch by intention!

cc @wing328 @jfiala

wing328 and others added 30 commits October 24, 2018 21:02
…1258)

* Port of PR swagger-api/swagger-codegen#8804.

* Correction of conflict with PR OpenAPITools#528 (missing template file).

* Add rust-reqwest samples to Circle CI tests.

* Add integration test pom.xml file with launcher to trigger cargo execution.

* Deduplicate Maven project name.

* Fix "api_key" header for Petstore.

* Better API key management.

* Fix query param for lists of objects other than strings (numbers, etc.).

* Update to reqwest 0.9, and refactor of header management (using reqwest transition feature).

* Merge scripts generating rust-hyper and rust-reqwest samples.

* Consistent full stops.

* Use raw variables in all Rust mustache templates.

* Replace production build in CI with a quick simple check.

* Update samples.

* Finish Reqwest 0.9 migration (removing "hyper 0.11" transition feature).

* Configuration implements Default trait.

* API template reorganized: HashMap is not required anymore.

* Revert "Merge scripts generating rust-hyper and rust-reqwest samples."

This reverts commit 970f996.

* Remove deprecated "-XX:MaxPermSize" java arg.
* handle and return API response decode error

* ensure-up-to-date

* refactor go/api.mustache for easier readability

* disable error reporting for TestPlaceOrder with invalid RFC 3339 `+0000` UTC offset present in Petstore test server
* fix wrong formParams type

* run typescript-angular-petstore-all.bat

* more typesafe fix

* fix formatting

* run typescript-angular-petstore-all.bat

* fix compile  problem with previous versions of angular

* generate code

* fix compile problem add <any>

* try Johannes Hoppe solution

* generate files
* Generator erlang-proper

Used to generate PropEr generators for property-based testing

* Fix binary/2 implementation. Add behaviour attribute.

* Remove line from copyright notice

* Avoid escpaing HTML and remove suffix from variable name

* Update samples

* Include querystring parameters

* We use export_all, don't consider warnings as errors

* List command sequence on failure

* Use hasConsumes instead

* Add nowarn_export_all, re-add warning_as_errors
* Improve method signatures to use const when the value won't change

* Update PetStore

* Change setters for non-primitive types to receive const reference parameters

* Update PetStore
* add group parameter support to php template

* more update for group parameter

* fix call to protect function

* fix missing $

* update based on feedback

* create fake endpoint to test group parameters

* sync fake petstore spec v2, v3

* fix php doc for group parameters

* update petstore samples

* update fake petstore spec v2

* update petstore samples

* fix spec and update samples
* prepare 3.3.2 release

* update script to remove outdated reference
* prepare 3.3.3-snapshot

* update petstore samples
* Convert floating point numbers to string with higher precision
* Update PetStore
* improve ensure-up-to-date script

* trigger build failure

* use exit 1 instead

* fix build failure

* update samples
* Add support for map
* Add support for nested maps
* Simplify Array and Map Helper
* Use const reference wherever possible
Add a note about use of `systemProperties` since some options are confusing and hard to guess how to use correctly. The only source I was able to find how to configure it in the way I need was a comment to issue OpenAPITools#551 (comment)
* Added hint for npm package wrapper to README

* Added manual how to use and install NPM package globally
…etry (OpenAPITools#1319)

* Add new ApiClient constructors for access token retry

* Update samples

* Update security samples
…APITools#1332)

* feat: OpenApi-generation from within eclipse (OpenAPITools#509)

* Added life-cycle-mapping for recognition by M2E
* Make BuildContext injectable by M2E in oder to detect if json-source
has been modified and a regeneration is required.

* core: fix indentation problems, remove commented code
james-rant and others added 19 commits November 10, 2018 13:59
* Fix warnings produced when using Swift 4.2

* Update Petstore client for Swift 4
* remove duplicated imports

* fix model import in java

* update generator docs

* fix model import for retrofit2 client
…ls#1415)

Add tip about running online openapi-generator via Docker
* add top level x-group-parameters support

* update petstore samples
* Fix return_type parameter examples

* $ bin/openapi3/ruby-client-petstore.sh
* test run-in-docker in travis

* fix command path

* correct batch mode arg

* disable useSystemClassLoader
…ls#1428)

* fix get schema type in abstract c# class

* update c# petstore sample
…es not import (OpenAPITools#1426)

* fix(OpenAPITools#1423): [JAVA] generating Map with jaxrs-reasteasy does not import
the hashmap

OpenAPITools#1423

* fix(OpenAPITools#1423): fix ensure-up-to-date issues for jaxrs-resteasy joda
samples

OpenAPITools#1423
…PITools#1438)

* remove model import when there is no model

* fix android http client no model issu due to import
* [csharp] Refactor to support third-party customization more easily

* [csharp] Regenerate OpenAPIClient sample

* create new csharp-refactor client gen

* update samples

* fix Locale.ROOT

* fix import

* remove outdated files, update samples
…ls#1440)

* add file post processing to cpp generators

* use clang to auto format cpp-restsdk code

* restore cpp-restsdk samples without clang format
wing328 and others added 6 commits November 14, 2018 23:56
Ensure the same date format string is used throughout the generated code (use the one set in Configuration.swift).

Ensure the same date formatter options are used when encoding dates as well as decoding dates. If a consumer has set their own date formatter on CodableHelper, use that when encoding dates too.

Adds DateFormatTests to the SWIFT4 unit tests.

Updates the SWIFT4 petstore samples
This reverts commit e5c0d22.
@renepardon renepardon closed this Nov 15, 2018
@renepardon renepardon deleted the graphql_client branch November 15, 2018 06:56
@wing328
Copy link
Member

wing328 commented Dec 6, 2018

Closed via #1603

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.