Skip to content

[DSM] Add technology pages for DSM documentation #28983

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 20 commits into
base: master
Choose a base branch
from

Conversation

lkretvix
Copy link

@lkretvix lkretvix commented Apr 25, 2025

What does this PR do? What is the motivation?

https://datadoghq.atlassian.net/browse/DSMON-685

Customers can find it difficult to find what technologies are supported by DSM. In order to do this they need to go to each language and look at the tech libraries there. Let's create an Inversion where users can view the technology and find the languages/libraries that are supported

Also some cleanup on duplicated sections of content, converting to shortcodes

QA

https://docs-staging.datadoghq.com/lucas.kretvix/dsm-add-new-tech-pages/data_streams/technologies/rabbitmq/

Merge instructions

Merge readiness:

  • Ready for merge

For Datadog employees:
Merge queue is enabled in this repo. Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). Without this format, your pull request will not pass in CI, the GitLab pipeline will not run, and you won't get a branch preview. Getting a branch preview makes it easier for us to check any issues with your PR, such as broken links.

If your branch doesn't follow this format, rename it or create a new branch and PR.

To have your PR automatically merged after it receives the required reviews, add the following PR comment:

/merge

Additional notes

@github-actions github-actions bot added the Architecture Everything related to the Doc backend label Apr 25, 2025
Comment on lines +13 to +30
<table>
<thead>
<tr>
<th>Language</th>
<th>Library</th>
<th>Minimal tracer version</th>
<th>Recommended tracer version</th>
</tr>
</thead>
<tbody>
<tr>
<td><a href="/data_streams/dotnet">.NET</a></td>
<td><a href="https://www.nuget.org/packages/Azure.Messaging.ServiceBus">Azure.Messaging.ServiceBus</a></td>
<td>2.53.0 </td>
<td>2.53.0 or later </td>
</tr>
</tbody>
</table>
Copy link
Author

Choose a reason for hiding this comment

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

decided to use html table styling for these for consistency. Some of the tech tables have cells that span multiple columns (kafka's Go language has multiple libraries for ex.)

Copy link
Contributor

Preview links (active after the build_preview check completes)

New or renamed files

Modified Files

@lkretvix lkretvix marked this pull request as ready for review April 25, 2025 16:35
@lkretvix lkretvix requested review from a team as code owners April 25, 2025 16:35
@jhgilbert jhgilbert added the editorial review Waiting on a more in-depth review label Apr 25, 2025
@johannbotha
Copy link
Contributor

image Thanks for adding these, but it looks like it doesn't scroll to the right section. It takes you to the main page, but the setup matrix is still past the fold.

@lkretvix
Copy link
Author

Thanks for adding these, but it looks like it doesn't scroll to the right section. It takes you to the main page, but the setup matrix is still past the fold.

@johannbotha Not sure if there's a way to force it. I'm adding the anchor tag there but it doesn't seem to want to scroll 🤔

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Architecture Everything related to the Doc backend editorial review Waiting on a more in-depth review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants