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.

Sign up for GitHub

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

Jump to bottom

[Provider] Openrouter, Lingyi, Moonshot and Zhipu #295

Merged
merged 20 commits into from
Apr 30, 2024
Merged

[Provider] Openrouter, Lingyi, Moonshot and Zhipu #295

merged 20 commits into from
Apr 30, 2024

Conversation

sternelee
Copy link
Contributor

Support openrouter.ai, lingyiwanwu.com and moonshot.cn, bigmodel.cn AI provider

VisargD
VisargD reviewed Apr 10, 2024
View reviewed changes
pnpm-lock.yaml Outdated Show resolved Hide resolved
VisargD
VisargD reviewed Apr 10, 2024
View reviewed changes
wrangler.toml Outdated Show resolved Hide resolved
@VisargD
Copy link
Collaborator

VisargD commented Apr 10, 2024

Hey @sternelee - Can you please also run npm run format to execute prettier formatting of the project. The PR check is failing due to it.

VisargD
VisargD requested changes Apr 10, 2024
View reviewed changes
Copy link
Collaborator

@VisargD VisargD left a comment

Choose a reason for hiding this comment

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

Hey @sternelee - Thanks for the PR! I have added first set of comments for the PR. Please check and let me know if you need any help with them

VisargD
VisargD reviewed Apr 10, 2024
View reviewed changes
wrangler.toml Outdated Show resolved Hide resolved
@sternelee sternelee requested a review from VisargD April 17, 2024 08:13
@sternelee
Copy link
Contributor Author

sternelee commented Apr 17, 2024 •
edited
Loading

@VisargD How do I modify the body structure？I want to adapted for reka.ai and coze.com bot

@VisargD
Copy link
Collaborator

VisargD commented Apr 23, 2024

Looks good to me. I will merge the PR in the next release. Thanks!

@VisargD
Copy link
Collaborator

VisargD commented Apr 29, 2024

Hey @sternelee - Looks like PR was updated and it shows 0 files changed in the diff now. Can you please check this once?

@sternelee
Copy link
Contributor Author

Hey @sternelee - Looks like PR was updated and it shows 0 files changed in the diff now. Can you please check this once?

oh, I don’t know why it automatically sync with the upstream. I have already pushed force it again.

@VisargD VisargD merged commit 8dd2602 into Portkey-AI:main Apr 30, 2024
1 check passed
@VisargD
Copy link
Collaborator

VisargD commented Apr 30, 2024

Closes #327
Closes #326
Closes #325
Closes #324

This was linked to issues Apr 30, 2024
[Provider]: Openrouter #324
Closed
[Provider]: Zhipu #325
Closed
[Provider]: Lingyi #326
Closed
[Provider]: Moonshot #327
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@unsync unsync unsync left review comments

@VisargD VisargD VisargD approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Provider]: Moonshot [Provider]: Lingyi [Provider]: Zhipu [Provider]: Openrouter
4 participants
@sternelee @VisargD @unsync @xingguang2024