Skip to content

Conversation

@NickLarsenNZ
Copy link
Member

  • warning level needs to be warn
    • I changed level to String, added off and referred the reader to external docs for more complex examples.
  • endpoint requires a scheme, otherwise the app logs errors.
    • Ideally this should be caught (or auto-prepended in the app).
  • I added backticks for each example value

@NickLarsenNZ NickLarsenNZ requested a review from Techassi November 4, 2025 12:26
@NickLarsenNZ NickLarsenNZ self-assigned this Nov 4, 2025
@netlify
Copy link

netlify bot commented Nov 4, 2025

Deploy Preview for stackable-docs ready!

Name Link
🔨 Latest commit af0cd3a
🔍 Latest deploy log https://app.netlify.com/projects/stackable-docs/deploys/6909f3e44bb4ec0008f2cff7
😎 Deploy Preview https://deploy-preview-794--stackable-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Techassi
Techassi previously approved these changes Nov 4, 2025
I chose not to move the title `[]` bit to the url definition so that it still reads well as text
@NickLarsenNZ NickLarsenNZ added this pull request to the merge queue Nov 4, 2025
Merged via the queue into main with commit 415d85e Nov 4, 2025
7 checks passed
@NickLarsenNZ NickLarsenNZ deleted the update-telemetry-docs branch November 4, 2025 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants