site admin: add link to Cody Analytics#60371
Conversation
chwarwick
left a comment
There was a problem hiding this comment.
Users will need to request access right? It's probably worth a note about how to do that.
(cherry picked from commit d99889d)
| }, | ||
| { | ||
| path: '/analytics/cody', | ||
| render: () => <AnalyticsCodyPage />, |
There was a problem hiding this comment.
Hey @taras-yemets , I know it's a bit late but we should be gating this path in the side-admin page.
This should only show up when the license has Cody enabled. I realize you have a backport PR and this is merged but do you think you can push a fix for this to main and another to the backport branch
There was a problem hiding this comment.
Nice catch, @BolajiOlajide! Thank you!
I ensured the page is not accessible via sidebar UI (https://github.com/sourcegraph/sourcegraph/pull/60371/commits/c71ffac46155b72378294bf113af234eb189322a), but missed to do the same on the route level.
Fixed in https://github.com/sourcegraph/sourcegraph/pull/60396.
I'll update the backport PR when the one to main is merged.
|
@taras-yemets here is the original screen I designed for this: I've replaced the image with a current snapshot of the design. Can we get that included so this page is a little less bare? Also, we need a way to link back to the instance on the analytics site. The analytics service seems to know about the instance as it reads sourcegraph.sourcegraph.com in the top right, but it doesn't link to the instance, and doesn't contain anything when there is once instance so it appears broken. |
Adds a link to Cody Analytics service.
Figma
Test plan