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

Implemented AI component #650

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

Implemented AI component #650

wants to merge 10 commits into from

Conversation

roll
Copy link
Collaborator

@roll roll commented Nov 18, 2024

@roll roll requested a review from romicolman November 18, 2024 14:58
@roll roll mentioned this pull request Nov 18, 2024
Copy link
Collaborator

@romicolman romicolman left a comment

Choose a reason for hiding this comment

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

HI @roll! I tried to test it but I can't paste the OpenAI key. Here is a video showing the problem:

Grabacion.de.pantalla.2024-11-21.a.la.s.7.41.47.a.m.mov

@roll
Copy link
Collaborator Author

roll commented Nov 25, 2024

@romicolman
Thanks, I see. It seems right-mouse clicks don't work in Electron for all the inputs in the app. I'll try to fix it

Copy link

cloudflare-workers-and-pages bot commented Nov 25, 2024

Deploying opendataeditor with  Cloudflare Pages  Cloudflare Pages

Latest commit: cd26709
Status: ✅  Deploy successful!
Preview URL: https://36b822fa.opendataeditor.pages.dev
Branch Preview URL: https://635-implement-ai-component-2.opendataeditor.pages.dev

View logs

@roll
Copy link
Collaborator Author

roll commented Nov 25, 2024

@romicolman
I fixed the problem -- actually, the application had never had any context menus before. So I had to enable it. You can try using the component.

@romicolman
Copy link
Collaborator

Hi @roll I tried to test it and I got this error:

Captura de pantalla 2024-11-25 a la(s) 2 23 43 p  m

Can you check if its working ok?

Steps:

  1. Added OpenAI key.
  2. The ODE displayed the message (prompt)
  3. Got the Unknown error message

@roll roll force-pushed the 635/implement-ai-component-2 branch from d13352e to d980936 Compare November 26, 2024 10:10
@roll
Copy link
Collaborator Author

roll commented Nov 26, 2024

@romicolman
I can't reproduce it but I have improved error indication -- can you please try again and share a new error message?

@roll
Copy link
Collaborator Author

roll commented Nov 26, 2024

BTW I would say that the most common way of calling this button is "AI Assistant" not just "AI"

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.

Implement AI component
2 participants