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

feat: Add ChatCompletions API #47

Merged
merged 4 commits into from
Mar 13, 2023
Merged

feat: Add ChatCompletions API #47

merged 4 commits into from
Mar 13, 2023

Conversation

osugikoji
Copy link
Contributor

@osugikoji osugikoji commented Mar 7, 2023 •
edited
Loading

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Creates chat completions entrypoint based on the following ticket: https://ymedialabs.atlassian.net/browse/CM-1219

  • Added sample for chat completions on JVM sample
  • Adjusted iOS and Android sample to use chat completions entrypoint instead of completion entrypoint
  • Added new unit tests

💚 How did you test it?

Tested on all platforms samples

📝 Checklist

  • I reviewed submitted code
  • I added tests to verify changes
  • All tests passing
  • All linters passing
  • There are no code climate issues

@osugikoji osugikoji self-assigned this Mar 7, 2023
@osugikoji osugikoji merged commit d34eb8f into main Mar 13, 2023
@osugikoji osugikoji deleted the feature/CM-1219/chat-completions branch March 13, 2023 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@renatoarg renatoarg renatoarg approved these changes

@kikoso kikoso Awaiting requested review from kikoso

Assignees

@osugikoji osugikoji

Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@osugikoji @renatoarg