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

[FEAT] Add Openrouter v2 #148

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

Conversation

charlwillia6
Copy link

Description ✏️

Details are here:
#35

This PR is replacing PR #35 . The original PR had too many changes that weren't related to the OpenRouter feature. This PR is simplified and has all the extra linting and configuration changes removed.

Screenshots are included in the closed PR #35

Checklist ✅

  • I have added screenshots (if UI changes are present).
  • I have done a self-review of my code.
  • I have manually tested my code (if applicable).

@jpan8866
Copy link

Are you able to run this locally? I'm not able to get a response due to 401 unauthorized. The apikey seems to not be saved, the value is undefined when sending the request

@charlwillia6
Copy link
Author

Are you able to run this locally? I'm not able to get a response due to 401 unauthorized. The apikey seems to not be saved, the value is undefined when sending the request

You're right! I had a mistyped variable (apikey should have been apiKey). I fixed it. Please try again.

The only issue I am struggling with at the moment is slash commands are not working. If you have any suggestions for that, I will push a fix.

@charlwillia6
Copy link
Author

image

@jpan8866
Copy link

Are you able to run this locally? I'm not able to get a response due to 401 unauthorized. The apikey seems to not be saved, the value is undefined when sending the request

You're right! I had a mistyped variable (apikey should have been apiKey). I fixed it. Please try again.

The only issue I am struggling with at the moment is slash commands are not working. If you have any suggestions for that, I will push a fix.

it seems that / command does not work with non-open router models as well. On main branch in debugger, i havent been able to make it work either

@charlwillia6
Copy link
Author

it seems that / command does not work with non-open router models as well. On main branch in debugger, i havent been able to make it work either

Hmm...I have been having problems with 1.5.0 also. Did this bug make it into production?

@charlwillia6
Copy link
Author

it seems that / command does not work with non-open router models as well. On main branch in debugger, i havent been able to make it work either

This bug with not being able to use Slash commands did make it into production. I have this issue with 1.5.0. I am raising an issue. So, I think this PR is unrelated to that issue and ready for merge if testing passes.

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