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

Add remix example #593

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Add remix example #593

wants to merge 5 commits into from

Conversation

nicnocquee
Copy link

This PR adds an example of using Remix with OpenAI. Should resolves issue #199

Copy link

@orca-security-us orca-security-us bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca

@MaxLeiter
Copy link
Member

Thanks! could you run prettier for fix up the CI? pnpm prettier-fix.

@nicnocquee
Copy link
Author

Thanks! could you run prettier for fix up the CI? pnpm prettier-fix.

I did it.

@MaxLeiter
Copy link
Member

Builds are failing

Error: Error loading Remix config at /home/runner/work/ai/ai/examples/remix-openai/remix.config.js
  SyntaxError: Unexpected token 'export'
      at Object.readConfig (/home/runner/work/ai/ai/node_modules/.pnpm/@remix-run+dev@2.0.1_@remix-run+serve@2.0.1_@types+node@17.0.12_typescript@5.1.3/node_modules/@remix-run/dev/dist/config.js:72:13)
      at async Object.build (/home/runner/work/ai/ai/node_modules/.pnpm/@remix-run+dev@2.0.1_@remix-run+serve@2.0.1_@types+node@17.0.12_typescript@5.1.3/node_modules/@remix-run/dev/dist/cli/commands.js:140:18)
      at async Object.run (/home/runner/work/ai/ai/node_modules/.pnpm/@remix-run+dev@2.0.1_@remix-run+serve@2.0.1_@types+node@17.0.12_typescript@5.1.3/node_modules/@remix-run/dev/dist/cli/run.js:176:7)
   ELIFECYCLE  Command failed with exit code 1.

@moltar
Copy link

moltar commented Mar 4, 2024

Has anyone gotten these examples to work?

All I am getting back as a response is the full HTML of the route.

@nicnocquee
Copy link
Author

I haven't got the time to fix the CI errors, sorry!

@laiso laiso mentioned this pull request Apr 4, 2024
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.

None yet

3 participants