vsce fix: remove credentials from graphQL & update endpoint
There are no commits yet
Push commits to the source branch or add previously merged commits to review them.
Created by: abeatrix
Close: https://github.com/sourcegraph/sourcegraph/issues/35955 & https://github.com/sourcegraph/sourcegraph/issues/35980
Issue summary: https://github.com/sourcegraph/sourcegraph/issues/35955
The integration test is currently failing for VSCE because of CORS issue caused by credentials: include
It was a leftover by mistake from my last PR when testing the streamlined auth flow.
Removing it has resolved the issue:
Issue summary: https://github.com/sourcegraph/sourcegraph/issues/35980
User is unable to login if they have submitted a new instance URL in the UI while the extension has already connected to the old instance URL (we use sourcegraph.com by default). This is because updating instance URL via the UI does not trigger extension to reload currently because we only update the instance URL in the User config if the auth call is completed successfully (code), and the reload will only be triggered if the instance URL has been changed in the config.
Update access tplem them instance URL in config after the form has been submitted to trigger extension to reload (code)
The test build from this branch is now passing the integration test.
Run yarn build:test
and then yarn test-integration
in the client/vscode
directory to run the integration test.
Check out the client app preview documentation to learn more.
Push commits to the source branch or add previously merged commits to review them.