Skip to content
This repository was archived by the owner on Jul 4, 2025. It is now read-only.

api: Update the API decorator #637

Merged
merged 1 commit into from
May 31, 2024
Merged

api: Update the API decorator #637

merged 1 commit into from
May 31, 2024

Conversation

@irfanpena
Copy link
Contributor

@irfanpena irfanpena commented May 31, 2024

Describe Your Changes

  • Remove the Cortex tag description because the Cortex endpoints are deprecated
  • Add desc for chat completions endpoint

Fixes Issues

  • Closes #
  • Closes #

Self Checklist

  • Added relevant comments, esp in complex areas
  • Updated docs (for bug fixes / features)
  • Created issues for follow-up changes or refactoring needed

@irfanpena irfanpena requested a review from louis-jan May 31, 2024 02:39
@irfanpena irfanpena self-assigned this May 31, 2024
@louis-jan
Copy link
Contributor

louis-jan commented May 31, 2024

Great! Thank you, please proceed

louis-jan
louis-jan approved these changes May 31, 2024
View reviewed changes
Copy link
Contributor

@louis-jan louis-jan left a comment

Choose a reason for hiding this comment

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

LGTM

@irfanpena irfanpena merged commit fe26363 into dev May 31, 2024
@irfanpena irfanpena deleted the api-update branch May 31, 2024 03:17
@irfanpena irfanpena restored the api-update branch May 31, 2024 03:17
@irfanpena irfanpena deleted the api-update branch July 29, 2024 06:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@louis-jan louis-jan louis-jan approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@irfanpena irfanpena

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@irfanpena @louis-jan