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

remove the v2 api route and associated code #185

Open

mme wants to merge 2 commits into main from remove-api-v2

Collaborator

mme commented Jan 25, 2024

This removes the v2 api route and the associated assistant API code.


          remove the v2 api route and associated code

a6b4f94

mme requested a review from ataibarkai

January 25, 2024 11:26

Collaborator Author

mme commented Jan 25, 2024

@ataibarkai please have a look if this is what we want.

This removes all the code associated with the assistant API too, since it depends on the v2 route.

We also discussed removing CopilotApiConfig but I think it is good to have this internally (to keep all configuration in one place)


          Merge branch 'main' into remove-api-v2

901203e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment