Skip to content

Conversation

@whitep4nth3r
Copy link
Contributor

Given that

  • /product/releases is one of the top most visited pages
  • when Googling "sentry sdk release" /product/releases ranks number 1 on Google (with the assumed intent that a developer wants to find out the latest release number for a given SDK)

I hypothesised that users were landing on this page whilst searching for SDK release information.

This PR adds a callout at the top of /product/releases to the Sentry GitHub profile.

Question for discussion:

Should we track if users are clicking on this link effectively to prove/disprove the hypothesis?
If we should, can we?

(This PR is in draft pending discussion.)

Priority

  • Urgent deadline (GA date, etc.):
  • Other deadline:
  • None: Not urgent, can wait up to 1 week+

@vercel
Copy link

vercel bot commented Oct 17, 2024

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

Name Status Preview Comments Updated (UTC)
changelog ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 31, 2024 2:43pm
develop-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 31, 2024 2:43pm
sentry-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 31, 2024 2:43pm

@codecov
Copy link

codecov bot commented Oct 17, 2024

Bundle Report

Changes will increase total bundle size by 46.42kB (0.33%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
sentry-docs-server-cjs 7.43MB 46.43kB (0.63%) ⬆️
sentry-docs-edge-server-array-push 333.11kB 3 bytes (-0.0%) ⬇️
sentry-docs-client-array-push 6.44MB 6 bytes (-0.0%) ⬇️

@whitep4nth3r
Copy link
Contributor Author

@lizokm I added the tracking link. It triggers stuff in the console (and is obviously ignored from localhost) but I think we're good to go! Please confirm that using release+cta as the event name is what you need for what is set up in Plausible.

Copy link
Contributor

@lizokm lizokm left a comment

Choose a reason for hiding this comment

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

LGTM!

@whitep4nth3r whitep4nth3r marked this pull request as ready for review November 1, 2024 08:04
@whitep4nth3r whitep4nth3r merged commit a40a687 into master Nov 1, 2024
11 checks passed
@whitep4nth3r whitep4nth3r deleted the add-sdk-release-callout branch November 1, 2024 08:04
@github-actions github-actions bot locked and limited conversation to collaborators Nov 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants