Skip to content

Add Analytics access token docs #1160

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 14 commits into from
May 29, 2025
Merged

Add Analytics access token docs #1160

merged 14 commits into from
May 29, 2025

Conversation

@jac
Copy link
Member

@jac jac commented May 28, 2025 •
edited
Loading

Adds documentation for access tokens in Sourcegraph Analytics.

@jac jac requested a review from MaedahBatool May 28, 2025 17:11
@vercel
Copy link

vercel bot commented May 28, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sourcegraph-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 29, 2025 6:04pm

@jac jac force-pushed the jac/core-1050 branch from 75680ce to c3acfd9 Compare May 28, 2025 17:17
bobheadxi
bobheadxi reviewed May 28, 2025
View reviewed changes
bobheadxi
bobheadxi reviewed May 28, 2025
View reviewed changes
bobheadxi
bobheadxi reviewed May 28, 2025
View reviewed changes
bobheadxi
bobheadxi reviewed May 28, 2025
View reviewed changes
bobheadxi
bobheadxi reviewed May 28, 2025
View reviewed changes
@jac jac marked this pull request as ready for review May 28, 2025 23:00
bobheadxi
bobheadxi reviewed May 28, 2025
View reviewed changes
bobheadxi
bobheadxi reviewed May 28, 2025
View reviewed changes
bobheadxi
bobheadxi reviewed May 28, 2025
View reviewed changes
bobheadxi
bobheadxi reviewed May 28, 2025
View reviewed changes
kalanchan
kalanchan reviewed May 28, 2025
View reviewed changes
Copy link
Contributor

@kalanchan kalanchan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we need to add a section that explains how to use the access token?

@jac
Copy link
Member Author

jac commented May 29, 2025

do we need to add a section that explains how to use the access token?

Added in 0b0b44d

@jac jac requested review from bobheadxi and kalanchan May 29, 2025 12:01
jac and others added 6 commits May 29, 2025 13:01
@jac
Add Analytics access token docs
4affb63
@jac
feedback
acfdaf3
@jac
session cookie screenshot
21c33ed
@jac @bobheadxi
Update docs/analytics/cloud.mdx
702ed30 
Co-authored-by: Robert Lin <robert@bobheadxi.dev>
@jac @bobheadxi
Update docs/analytics/cloud.mdx
bbb84f7 
Co-authored-by: Robert Lin <robert@bobheadxi.dev>
@jac @bobheadxi
Update docs/analytics/cloud.mdx
43a93cc 
Co-authored-by: Robert Lin <robert@bobheadxi.dev>
jac and others added 3 commits May 29, 2025 13:01
@jac jac force-pushed the jac/core-1050 branch from 0b0b44d to ef3379e Compare May 29, 2025 12:01
bobheadxi
bobheadxi reviewed May 29, 2025
View reviewed changes
bobheadxi
bobheadxi reviewed May 29, 2025
View reviewed changes
bobheadxi
bobheadxi reviewed May 29, 2025
View reviewed changes
jac and others added 2 commits May 29, 2025 18:29
@jac @bobheadxi
Update docs/analytics/cloud.mdx
c5ab5ab 
Co-authored-by: Robert Lin <robert@bobheadxi.dev>
@jac @bobheadxi
Apply suggestions from code review
c8ecda5 
Co-authored-by: Robert Lin <robert@bobheadxi.dev>
MaedahBatool
MaedahBatool approved these changes May 29, 2025
View reviewed changes
Copy link
Contributor

@MaedahBatool MaedahBatool left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @jac, added a few improvements and callouts to your branch. Good to go live. ✅

@jac jac merged commit 1a53a75 into main May 29, 2025
5 checks passed
@jac jac deleted the jac/core-1050 branch May 29, 2025 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bobheadxi bobheadxi bobheadxi approved these changes

@federubinstein federubinstein Awaiting requested review from federubinstein

@kalanchan kalanchan Awaiting requested review from kalanchan

+1 more reviewer

@MaedahBatool MaedahBatool MaedahBatool approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@jac @MaedahBatool @bobheadxi @kalanchan