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

VSCode: Update Cody feedback URL to Sourcegraph Community forum #5936

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

PriNova
Copy link
Collaborator

@PriNova PriNova commented Oct 17, 2024

This PR updates the CODY_FEEDBACK_URL in the VSCode extension to redirect users to the Sourcegraph Community forum instead of GitHub issues. This change aims to improve user engagement and streamline the feedback process by leveraging a more collaborative platform.

Changes

  • Modified vscode/src/chat/protocol.ts to update the CODY_FEEDBACK_URL constant
  • Old URL: https://github.com/sourcegraph/cody/issues/new/choose
  • New URL: https://community.sourcegraph.com/new-topic?title=&body=&category=cody&tags=feedback,vs-code

Benefits

  • Centralizes feedback in the Sourcegraph Community forum
  • Encourages user interaction and discussion
  • Simplifies the process of tracking and addressing user feedback
  • Aligns with our community-driven approach to product improvement

Test plan

  • Verified that the new URL is correctly set in the codebase
  • Tested the link functionality in the VSCode extension

Changelog

Change the CODY_FEEDBACK_URL from GitHub issues to the Sourcegraph Community forum. This redirects user feedback to a more collaborative platform, potentially improving user engagement and streamlining the feedback process.

- Old URL: https://github.com/sourcegraph/cody/issues/new/choose
- New URL: https://community.sourcegraph.com/new-topic?title=&body=&category=cody&tags=feedback,vs-code
@PriNova PriNova requested a review from jdorfman October 17, 2024 17:03
Copy link
Member

@jdorfman jdorfman left a comment

Choose a reason for hiding this comment

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

LGTM!

@jdorfman
Copy link
Member

@kalanchan can you give this a stamp and merge

Copy link
Contributor

@kalanchan kalanchan left a comment

Choose a reason for hiding this comment

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

gogogo!

@jdorfman
Copy link
Member

@kalanchan @PriNova I'm not sure how to make this pass.

@PriNova
Copy link
Collaborator Author

PriNova commented Oct 22, 2024

@jdorfman Have you write permissions to press the merge button with a force-merge? I don't know, how it is internally handled, but I guess a squash will do it.

@jdorfman
Copy link
Member

I don't (by design). I'll work with Kalan on getting it merged tomorrow.

@PriNova
Copy link
Collaborator Author

PriNova commented Oct 24, 2024

can be merged now @kalanchan

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.

3 participants