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

Fix context files urls #4955

Merged

mkondratek merged 1 commit into main from mkondratek/fix/context-file-urls

Jul 20, 2024

Contributor

mkondratek commented Jul 20, 2024 •

edited

Loading

A recent change brok the urls. This PR reverts the original behaviour.

The recent change: #4706 (comment)
Slack thread: https://sourcegraph.slack.com/archives/C05AGQYD528/p1721382757890889

Test plan

chat context file items looks properly for remote files


          Fix context files urls

f94dd2a

mkondratek self-assigned this

mkondratek mentioned this pull request

Update Cody commit sourcegraph/jetbrains#1927

Merged

RXminuS approved these changes

View reviewed changes

mkondratek merged commit 46b3f10 into main

21 checks passed

mkondratek deleted the mkondratek/fix/context-file-urls branch

July 20, 2024 11:47

pkukielka pushed a commit to sourcegraph/jetbrains that referenced this pull request


          Update Cody commit (#1927)

3ce3836

This PR update's cody commit. The most important LLM selection dropdown
**with self hosted models** is expected to be visible for enterprise
users now.

The important PRs from Cody to be included:
- sourcegraph/cody#4913 (self hosted models)
- ~sourcegraph/cody#4706 ([a bug with
paths](sourcegraph/cody#4706 (comment))?
[slack
thread](https://sourcegraph.slack.com/archives/C05AGQYD528/p1721382757890889))~
FIXED BY sourcegraph/cody#4955

## Test plan
1. full QA

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

Labels

None yet