8000 Changed sankeys content to pathways by iadjivon · Pull Request #29229 · DataDog/documentation · GitHub
[go: up one dir, main page]
Skip to content

Changed sankeys content to pathways #29229

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
May 19, 2025
Merged

Changed sankeys content to pathways #29229

merged 3 commits into from
May 19, 2025

Conversation

iadjivon
Copy link
Contributor
@iadjivon iadjivon commented May 7, 2025 •
edited
Loading

What does this PR do? What is the motivation?

The PANA UI no longer mentions Sankeys. It has been changed to Pathways. Changing the Docs to reflect this.

Merge instructions

Merge readiness:

  • Ready for merge

@iadjivon iadjivon requested a review from a team as a code owner May 7, 2025 21:54
@github-actions github-actions bot added the Images Images are added/removed with this PR label May 7, 2025
@github-actions GitHub Actions
Copy link
Contributor
github-actions bot commented May 7, 2025

Preview links (active after the build_preview check completes)

Modified Files

@buraizu buraizu self-assigned this May 7, 2025
buraizu
buraizu reviewed May 7, 2025
View reviewed changes
Copy link
Contributor
@buraizu buraizu left a comment

Choose a reason for hiding this comment

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

Looks good overall, just left one suggestion and a question

@iadjivon iadjivon requested a review from buraizu May 14, 2025 22:09
@iadjivon @buraizu
Update content/en/product_analytics/segmentation/_index.md
b66daed 
Co-authored-by: Bryce Eadie <bryce.eadie@datadoghq.com>
buraizu
buraizu approved these changes May 16, 2025
View reviewed changes
Copy link
Contributor
@buraizu buraizu left a comment

Choose a reason for hiding this comment

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

@iadjivon thanks! I just realized that the text above the image has the same issue, so I'm approving with one final suggestion to correct that.

@iadjivon @buraizu
Update content/en/product_analytics/segmentation/_index.md
86a23b9 
Co-authored-by: Bryce Eadie <bryce.eadie@datadoghq.com>
@iadjivon
Copy link
Contributor Author

/merge

@dd-devflow
Copy link
dd-devflow bot commented May 19, 2025 •
edited
Loading

View all feedbacks in Devflow UI.

2025-05-19 19:42:30 UTC ℹ️ Start processing command /merge

2025-05-19 19:42:41 UTC ℹ️ MergeQueue: waiting for PR to be ready

This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.

2025-05-19 20:10:15 UTC ℹ️ MergeQueue: merge request added to the queue

The expected merge time in master is approximately 21m (p90).

2025-05-19 20:30:12 UTC ℹ️ MergeQueue: This merge request was merged

@dd-mergequeue dd-mergequeue bot merged commit 748f73d into master May 19, 2025
17 of 21 checks passed
@dd-mergequeue dd-mergequeue bot deleted the ida.adjivon/DOCS-10806-sankey-to-pathways branch May 19, 2025 20:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@buraizu buraizu buraizu approved these changes

Assignees

@buraizu buraizu

Labels
Images Images are added/removed with this PR mergequeue-status: done
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@iadjivon @buraizu
0