-
Notifications
You must be signed in to change notification settings - Fork 248
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
Add support link for Cody Pro & Enterprise #3330
Conversation
@enriquegh it would be good to pre-fill the form with the current Cody for VS Code version automatically — is it possible to pass it URL params to https://help.sourcegraph.com/hc/en-us/requests/new? We could pass the version number and account ID, account type, etc. |
@abeatrix could I please get your assistance making the display of the quickpick item conditional, similar to how it works in the treeview? |
@abeatrix doh, I forget to check in the code that needed the conditionals! Fixed in d8b96dd53 |
d8b96dd
to
82b452f
Compare
@toolmantim pushed my changes to the branch 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome!
It would be nice to add a e2e test for this, but we can do it in a follow-up PR 🎉
Added some e2e tests but something funky is going on with the unit tests now 🤔 |
That's probably related to an issue @valerybugakov is working on |
Yeah, unit tests should be fixed on main now 👍 |
I think the changes in |
Adds Support to the sidebar and settings quickpick:
Test plan