refactor: logs integration docs clarity#16445
Merged
Conversation
Removed the "Integrations" section from various platform logging documentation files and replaced it with "Other Logging Integrations" to enhance clarity and consistency. This change was applied to the Android, Dart, .NET, Go, Java, PHP, Python, React Native, and Ruby platforms. Additionally, updated the Go integrations file to change the heading from "Supported libraries" to "Available integrations" for better alignment with the new structure.
Replaced plain text links with the new IntegrationListItem component for the Dart, .NET, Go, Java, and PHP integration documentation. This change enhances consistency and improves the overall structure of the integration listings across multiple platforms.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Hides the current page from the 'Other Logging Integrations' list on platform logs docs so users don't see a link to the page they're on. Co-Authored-By: Claude <noreply@anthropic.com> Co-authored-by: Cursor <cursoragent@cursor.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
Modified the Go integration documentation to include updated paths for Slog and Logrus links, ensuring they point to the correct log sections. Additionally, removed the unused IntegrationLink component from the mdxComponents file to streamline imports.
coolguyzone
approved these changes
Feb 18, 2026
Contributor
coolguyzone
left a comment
There was a problem hiding this comment.
Checked the content and the functionality of the new component, looks good to me!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
On the majority of SDK log pages, there is a section in the middle titled "Integrations". This is misleading, as it seems as if it could be referring to integrations relevant to the current page, when in reality, it is referring to other integrations alternative to the current page. This list would also include the current page as an item in the list.
This PR renames this section to
Other Logging Integrations, moves it to the bottom of the page, and excludes the current page from the list.Old: https://docs.sentry.io/platforms/java/guides/log4j2/logs/
New: https://sentry-docs-git-refactor-logsdocs.sentry.dev/platforms/java/guides/log4j2/logs/
Documentation structure and content improvements:
index.mdxfiles for Android, Dart, .NET, Go, Java, PHP, Python, Ruby, React Native). [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12]Integration list formatting standardization:
<IntegrationListItem>component within an unordered list, replacing plain markdown links for improved consistency and styling. [1] [2] [3] [4] [5]Component and code updates:
IntegrationLinkandIntegrationListItemcomponents to the MDX components registry to support the new integration list formatting. [1] [2]