Skip to content

Conversation

@lizokm
Copy link
Contributor

@lizokm lizokm commented Oct 30, 2024

No description provided.

@vercel
Copy link

vercel bot commented Oct 30, 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 Nov 11, 2024 5:53pm
sentry-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 11, 2024 5:53pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
develop-docs ⬜️ Ignored (Inspect) Visit Preview Nov 11, 2024 5:53pm

@codecov
Copy link

codecov bot commented Oct 30, 2024

Bundle Report

Changes will decrease total bundle size by 15 bytes (-0.0%) ⬇️. This is within the configured threshold ✅

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


<Alert>
Check out this video on [**Migrating to Sentry SaaS**](https://sentry.io/resources/migrate-to-sentry-saas-workshop/) to learn about our relocation tooling.
Check out this video on [**Migrating to Sentry SaaS**](<a href="https://sentry.io/resources/migrate-to-sentry-saas-workshop/" className="plausible-event-name=migrating+to+saas+video">Migrating to Sentry SaaS</a>) to learn about our relocation tooling.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
Check out this video on [**Migrating to Sentry SaaS**](<a href="https://sentry.io/resources/migrate-to-sentry-saas-workshop/" className="plausible-event-name=migrating+to+saas+video">Migrating to Sentry SaaS</a>) to learn about our relocation tooling.
Check out this video on [**Migrating to Sentry SaaS**](<a href="https://github.com/getsentry" className="plausible-event-name=release+health”>Sentry's GitHub profile</a>) to learn about our relocation tooling.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@whitep4nth3r apparently this is all you need :)

@lizokm lizokm requested a review from coolguyzone October 30, 2024 19:50
@coolguyzone
Copy link
Contributor

Hi @lizokm , I just checked this out on https://sentry-docs-git-add-plasuible-tracking-link-to-video.sentry.dev/concepts/migration/ and am seeing the following:

Screenshot 2024-10-30 at 1 48 22 PM

Looks like it's not quite right yet

Copy link
Contributor

@coolguyzone coolguyzone left a comment

Choose a reason for hiding this comment

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

Good to go!

@coolguyzone coolguyzone self-requested a review October 31, 2024 05:28
@lizokm
Copy link
Contributor Author

lizokm commented Oct 31, 2024

@coolguyzone thank you for catching that! That comment was supposed to go on a different PR!

Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
@lizokm lizokm merged commit 102f0ff into master Nov 11, 2024
11 checks passed
@lizokm lizokm deleted the add-plasuible-tracking-link-to-video branch November 11, 2024 18:01
vaind pushed a commit that referenced this pull request Nov 11, 2024
* Add plausible tracking

* Update docs/concepts/migration/index.mdx

* Update docs/concepts/migration/index.mdx

Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>

---------

Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
@github-actions github-actions bot locked and limited conversation to collaborators Nov 27, 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