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

🌿 Fern Regeneration -- July 22, 2024 #6

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

fern-api[bot]
Copy link
Contributor

@fern-api fern-api bot commented Jul 22, 2024

This PR regenerates code to match the latest API Definition.

@fern-api fern-api bot requested a review from fern-support July 22, 2024 19:21
Comment on lines +21 to +24
const client = new GooeyClient({ authorization: "YOUR_AUTHORIZATION" });
await client.copilotIntegrations.videoBotsStreamCreate({
integrationId: "integration_id",
});
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

usage should be more descriptive and have some use case that is demonstrated

@nikochiko
Copy link
Member

reference.md has the same problem that we solved with preferred_fields in gooey-server.

e.g. AI QR code only has a text prompt and no URL in the reference.

@nikochiko
Copy link
Member

nikochiko commented Jul 26, 2024

Edit: works now

@fern-bot

$ pnpm exec node index.js

node:internal/modules/cjs/loader:408
      throw err;
      ^

Error: Cannot find module '/Users/kaustubh/work/gooey-ts-test/node_modules/gooey/index.js'. Please verify that the package.json has a valid "main" entry
    at tryPackage (node:internal/modules/cjs/loader:400:19)
    at Module._findPath (node:internal/modules/cjs/loader:613:18)
    at Module._resolveFilename (node:internal/modules/cjs/loader:981:27)
    at Module._load (node:internal/modules/cjs/loader:841:27)
    at Module.require (node:internal/modules/cjs/loader:1061:19)
    at require (node:internal/modules/cjs/helpers:103:18)
    at Object.<anonymous> (/Users/kaustubh/work/gooey-ts-test/index.js:39:15)
    at Module._compile (node:internal/modules/cjs/loader:1159:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1213:10)
    at Module.load (node:internal/modules/cjs/loader:1037:32) {
  code: 'MODULE_NOT_FOUND',
  path: '/Users/kaustubh/work/gooey-ts-test/node_modules/gooey/package.json',
  requestPath: 'gooey'
}

Node.js v18.12.0

This is my package.json:

{
  "name": "gooey-ts-test",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "keywords": [],
  "author": "",
  "license": "ISC",
  "devDependencies": {
    "ts-node": "^10.9.2",
    "typescript": "^5.5.4"
  },
  "dependencies": {
    "gooey": "link:../typescript-sdk"
  }
}

@nikochiko
Copy link
Member

The authorization param should just accept the API key.

Right now, it expects Bearer $api_key. The bearer part should be auto built.

@nikochiko
Copy link
Member

reference.md should have information about the response formats. It doesn't have the expanded type info of responses (and requests too?)

@dsinghvi
Copy link

@nikochiko thanks for the feedback

usage should be more descriptive and have some use case that is demonstrated

The usage examples come from your OpenAPI spec, would you be open to adding some more examples to your OpenAPI spec or linking us to examples that you would want demonstrated?

reference.md has the same problem that we solved with preferred_fields in gooey-server.

Do you mind providing us more context here. We have a slack channel with y'all + happy to hop on a call to understand.

The authorization param should just accept the API key.

This is a change we can make to your OpenAPI spec (by specifying the bearer scheme)

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.

2 participants