Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Update Cody Web to 0.3.7#64296

Merged
0xnmn merged 5 commits into
mainfrom
naman/update-cody-web-version
Aug 7, 2024
Merged

Update Cody Web to 0.3.7#64296
0xnmn merged 5 commits into
mainfrom
naman/update-cody-web-version

Conversation

@0xnmn

@0xnmn 0xnmn commented Aug 6, 2024 •
edited
Loading

Copy link
Copy Markdown
Contributor

closes: https://linear.app/sourcegraph/issue/SRCH-821/context-is-not-being-fetched-on-vs-code-even-though-its-being-fetched

CleanShot 2024-08-06 at 16 13 05@2x

This PR updates Cody Web to 0.3.7. The latest version introduces the following changes:

  • Removes the Cody history panel and rather uses the tabs. (As updated in latest version on Cody Web)
  • Removes the experimental.noodle flag set to true.
  • Fixes the issue where the query for getCodyContext contained mention chips text and was different from VS Code.

Test plan

  • visit /cody/chat

Changelog

@cla-bot cla-bot Bot added the cla-signed label Aug 6, 2024
@0xnmn 0xnmn force-pushed the naman/update-cody-web-version branch from afd8692 to 17e0f86 Compare August 6, 2024 16:31
@0xnmn 0xnmn requested a review from a team August 6, 2024 17:21
peterguy
peterguy approved these changes Aug 6, 2024
View reviewed changes

@peterguy peterguy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

works great for me locally.

@fkling

fkling commented Aug 6, 2024

Copy link
Copy Markdown
Contributor

Does this also affect the behavior of the cody sidebar? I assume so. In that case the svelte app needs to be updated too.

@0xnmn

0xnmn commented Aug 6, 2024

Copy link
Copy Markdown
Contributor Author

@fkling yes I missed the svelte one. Working on it now. Thanks for catching it.

@0xnmn 0xnmn force-pushed the naman/update-cody-web-version branch 2 times, most recently from 0251112 to 44adaa2 Compare August 7, 2024 04:25
@0xnmn

0xnmn commented Aug 7, 2024 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

@fkling I have updated the Svelte Code as well. It is a small change, just 2 import renames but I am not able to test it locally.

Have spent enough time figuring out how to solve this 500 issue on sg start. I am getting this error even on main HEAD.
CleanShot 2024-08-07 at 10 09 58@2x

With pnpm dev for svelte and sg start web-standalone I am not able to login locally.

Given this is an easily reversible & small change, I am merging the PR and triggering a dotcom deployment to test it on prod. While I wait for team members to start working today, so I can get help to test it locally as well.

@0xnmn 0xnmn force-pushed the naman/update-cody-web-version branch from 44adaa2 to a9f04e2 Compare August 7, 2024 04:50
@0xnmn 0xnmn merged commit 3680503 into main Aug 7, 2024
@0xnmn 0xnmn deleted the naman/update-cody-web-version branch August 7, 2024 04:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@peterguy peterguy peterguy approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cla-signed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@0xnmn @fkling @peterguy