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

Migrate remaining endpoints (with a single exception) #2634

Closed
wants to merge 6 commits into from

Conversation

mkondratek
Copy link
Contributor

@mkondratek mkondratek commented Nov 13, 2024

This PR is a part of the protocol migration. Some endpoints are written by hand. We are switching to the protocol generated from Cody. In this PR:

Based on PR #2633

Full chain of PRs as of 2024-11-13

Test plan

  • All migrated endpoints have been verified with debugger (and reviewed in the trace log).
  • New chat action executed 🟢
  • Autocomplete tested 🟢
  • Inline Edits (including cancel, undo, retry, accept) tested 🟢
  • Chat tested 🟢
  • Closing/Opening the project, closing/opening files, closing/opening new chats tested 🟢
  • Swtiching account (dotcom -> enterprsise) tested 🟢
  • Cody Ignore (with CODY_JETBRAINS_FEATURES=cody.feature.internals-menu=true) 🟢

@mkondratek mkondratek self-assigned this Nov 13, 2024
@mkondratek mkondratek changed the title Migrate editTask/* Migrate remaining endpoints (with a single exception) Nov 13, 2024
@mkondratek mkondratek force-pushed the mkondratek/chore/migrate-api-part-2 branch 2 times, most recently from 673dacb to f20f828 Compare November 13, 2024 10:20
@mkondratek mkondratek force-pushed the mkondratek/chore/migrate-api-part-3 branch from dde28c8 to 535757e Compare November 13, 2024 10:22
Base automatically changed from mkondratek/chore/migrate-api-part-2 to main November 13, 2024 10:33
@mkondratek mkondratek force-pushed the mkondratek/chore/migrate-api-part-3 branch from 535757e to 3d39924 Compare November 13, 2024 10:34
@mkondratek
Copy link
Contributor Author

@mkondratek mkondratek closed this Nov 13, 2024
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.

1 participant