Skip to content

Conversation

@Lms24
Copy link
Member

@Lms24 Lms24 commented Oct 15, 2024

This PR improves the SvelteKit Cloudflare guide code block a bit by

  • fixing formatting of the diff block (TIL, the code block language needs to be set to <language> diff)
  • re-adding the handleError hook just to point out that this remains unchanged from the default setup. (@AbhiPrasad Just to sanity-check: It does remain unchanged, right?)

@vercel
Copy link

vercel bot commented Oct 15, 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 15, 2024 8:23am
sentry-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 15, 2024 8:23am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
develop-docs ⬜️ Ignored (Inspect) Visit Preview Oct 15, 2024 8:23am

@codecov
Copy link

codecov bot commented Oct 15, 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.46MB 6 bytes (-0.0%) ⬇️
sentry-docs-edge-server-array-push 257.08kB 3 bytes (-0.0%) ⬇️
sentry-docs-client-array-push 6.43MB 6 bytes (-0.0%) ⬇️

Copy link
Member

@AbhiPrasad AbhiPrasad left a comment

Choose a reason for hiding this comment

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

Yes the error handler should remain unchanged - thanks for the fix!

@coolguyzone
Copy link
Contributor

Hi @Lms24, thanks for the update! I noticed that the example code here is for a file that could be a .ts or .js file. The code example provided is in TypeScript, should we also provide the alternative in JavaScript? Or do you think the TS example will suffice?

@Lms24
Copy link
Member Author

Lms24 commented Oct 17, 2024

@coolguyzone the code for JS is identical (minus any type syntax but in this example there's none). I think for the moment, the TS example should suffice and we can spare us the duplication.

@Lms24 Lms24 merged commit 62bf803 into master Oct 17, 2024
11 checks passed
@Lms24 Lms24 deleted the lms/fix-cf-sveltekit-code branch October 17, 2024 07:20
@github-actions github-actions bot locked and limited conversation to collaborators Nov 2, 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.

4 participants