Skip to content

web: remove integrations banners and corresponding pings

Warren Gifford requested to merge taras-yemets/remove-extensions-banners into main

Created by: taras-yemets

Closes https://github.com/sourcegraph/sourcegraph/issues/38625

Removes browser and IDE extensions banners from Sourcegraph website and corresponding pings.

Removed banners (screenshots in this comment):

  • Search results page
    • Browser extension, IDE, and signup banners
  • File page
    • Browser extension banner (and tooltip preceding it) shown when hovering on the 'View on the code host' link with a certain cadence
  • VSCode extension
    • Signup banner

Reverts changes introduced by the following PRs:

  • #30169
  • #30400
  • #29966
  • #31389
  • #14256 (partially)

Test plan

  • ensure tests un CI pass
  • sg start web-standalone
  • Sourcegraph web
    • on search results page
      • reload it not less than five times and ensure that neither browser extension nor IDE extensions banner is shown
      • on sourcegraph.com: ensure you're not logged in and the signup banner is not visible (try multiple page reloads too)
    • on file page
      • hover on different code tokens not less than five times, then hover over View on the code host icon button and ensure that the browser extension popup is not shown (click should take you to the code host)
  • VSCode extension
    • search results view
      • on sourcegraph.com: ensure you're not logged in and the signup banner is not visible

App preview:

Check out the client app preview documentation to learn more.

Merge request reports

Loading