Remove subOperations and fix link

See #188 #198

Signed-off-by: Alan Cha <[email protected]>

Author: Alan-Cha

README.md

@@ -60,7 +60,7 @@ Here are some guides to further help you get started:
   <img src="https://raw.githubusercontent.com/ibm/openapi-to-graphql/master/docs/data-centric.png" alt="Example of data-centric design" width="600">
 
 - **Nested data**
-  [Links](https://github.com/OAI/OpenAPI-Specification/blob/OpenAPI.next/versions/3.0.md#linksObject) defined in the OAS are used to compose data definitions. Furthermore, hierarchical path structures can be used to nest data via the [`addSubOperations`](./packages/openapi-to-graphql/README.md#options-addsuboperations) option.
+  [Links](https://github.com/OAI/OpenAPI-Specification/blob/OpenAPI.next/versions/3.0.0.md#link-Object) defined in the OAS are used to create nested data structures, allowing for (deeply) nested queries. 
 
   <img src="https://raw.githubusercontent.com/ibm/openapi-to-graphql/master/docs/links.png" alt="Example of links resolution" width="600">
 

packages/openapi-to-graphql/README.md

@@ -179,7 +179,6 @@ OpenAPI-to-GraphQL.createGraphQLSchema(oas, {
   qs: {
     limit: 30 // send limit query string in every request
   },
-  addSubOperations: false,
   requestOptions: {
     proxy: "http://my-proxy:3128"
   },

packages/openapi-to-graphql/lib/types/operation.d.ts

@@ -80,10 +80,6 @@ export declare type Operation = {
      * (Local) server definitions of the operation.
      */
     servers: ServerObject[];
-    /**
-     * List of operations which are nested based on their path.
-     */
-    subOps?: Operation[];
     /**
      * Whether this operation should be placed in an authentication viewer\
      * (cannot be true if "viewer" option passed to OpenAPI-to-GraphQL is false).

packages/openapi-to-graphql/src/types/operation.ts

@@ -117,11 +117,6 @@ export type Operation = {
    */
   servers: ServerObject[]
 
-  /**
-   * List of operations which are nested based on their path.
-   */
-  subOps?: Operation[]
-
   /**
    * Whether this operation should be placed in an authentication viewer\
    * (cannot be true if "viewer" option passed to OpenAPI-to-GraphQL is false).

packages/openapi-to-graphql/test/evaluation/eval_apis_guru.js

@@ -51,8 +51,7 @@ async function checkOas(OASList) {
     console.log(`Process "${name}" (${oas['x-file-path']})...`)
     try {
       let { report } = await openapiToGraphql.createGraphQlSchema(oas, {
-        strict: false,
-        addSubOperations: false
+        strict: false
       })
       results.successes.push({ name, report })
     } catch (error) {

packages/openapi-to-graphql/test/example_api.test.js

@@ -24,7 +24,6 @@ beforeAll(() => {
   return Promise.all([
     openapiToGraphql
       .createGraphQlSchema(oas, {
-        addSubOperations: true,
         fillEmptyResponses: true
       })
       .then(({ schema, report }) => {

packages/openapi-to-graphql/test/example_api3.test.js

@@ -28,7 +28,6 @@ beforeAll(() => {
   return Promise.all([
     openapiToGraphql
       .createGraphQlSchema([oas, oas3], {
-        addSubOperations: true,
         fillEmptyResponses: true
       })
       .then(({ schema, report }) => {