Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Render Cody analytics site admin route if Cody is enabled in license#60396

Merged
taras-yemets merged 1 commit into
mainfrom
ty/add-cody-analytics-route-condition
Feb 12, 2024
Merged

Render Cody analytics site admin route if Cody is enabled in license#60396
taras-yemets merged 1 commit into
mainfrom
ty/add-cody-analytics-route-condition

Conversation

@taras-yemets

Copy link
Copy Markdown
Contributor

Follow-up on https://github.com/sourcegraph/sourcegraph/pull/60371 (comment).

Cody feature enabled in license
Screenshot 2024-02-12 at 08 30 03

Cody feature disabled in license
Screenshot 2024-02-12 at 08 27 39

Test plan

  • CI
  • Tested manually: Cody analytics page is not available in the site admin sidebar and via URL "site-admin/analytics/cody"

@cla-bot cla-bot Bot added the cla-signed label Feb 12, 2024
@taras-yemets taras-yemets requested review from a team and BolajiOlajide February 12, 2024 06:33
@taras-yemets taras-yemets enabled auto-merge (squash) February 12, 2024 06:37
@taras-yemets taras-yemets merged commit b6c40fc into main Feb 12, 2024
@taras-yemets taras-yemets deleted the ty/add-cody-analytics-route-condition branch February 12, 2024 06:46
keegancsmith pushed a commit that referenced this pull request Feb 12, 2024
* site admin: add link to Cody Analytics (#60371)

(cherry picked from commit d99889d)

* Render Cody analytics site admin route if Cody is enabled in license (#60396)

---------

Co-authored-by: Taras Yemets <yemets.taras@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants