Skip to content

✨ RUM Browser Angular Integration#2940

Merged
steveny91 merged 5 commits intomasterfrom
romanG/angular-integration-docs
Apr 22, 2026
Merged

✨ RUM Browser Angular Integration#2940
steveny91 merged 5 commits intomasterfrom
romanG/angular-integration-docs

Conversation

@rgaignault
Copy link
Copy Markdown
Contributor

@rgaignault rgaignault commented Mar 23, 2026

What does this PR do?

Updates the Angular integration tile with documentation for the new @datadog/browser-rum-angular plugin, covering error tracking and router integration.

Motivation

The @datadog/browser-rum-angular package is being released with two features:

The existing Angular tile only had generic RUM/logs/traces links. This PR adds plugin-specific documentation following the same structure as the React integration.

Review checklist

  • PR has a meaningful title or PR has the no-changelog label attached
  • Feature or bugfix has tests
  • Git history is clean
  • If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo

@rgaignault rgaignault requested a review from a team as a code owner March 23, 2026 12:56
@rgaignault rgaignault requested a review from guilhermevrs March 23, 2026 12:56
@rgaignault rgaignault requested a review from a team as a code owner March 23, 2026 13:03
@rgaignault rgaignault marked this pull request as draft March 23, 2026 13:28
@amortemousque amortemousque changed the title Add documentation for the RUM Angular integration plugin ✨ RUM Browser Angular Integration Apr 10, 2026
@amortemousque amortemousque marked this pull request as ready for review April 10, 2026 07:26
@rtrieu rtrieu added the editorial review Waiting on a more in-depth review from a docs team editor label Apr 10, 2026
@rtrieu
Copy link
Copy Markdown
Contributor

rtrieu commented Apr 10, 2026

Created DOCS-13971 for editorial review.

Copy link
Copy Markdown
Contributor

@iadjivon iadjivon left a comment

Choose a reason for hiding this comment

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

Hi there,
Thanks for your patience here!
I've added a couple of suggestions and recommendation.
Approved as to not be a blocker on this, but recommend reviewing the suggestions. Thanks so much and please let me know if you have any questions!

Comment thread rum_angular/README.md Outdated
Comment thread rum_angular/README.md Outdated
Comment thread rum_angular/README.md Outdated
Comment thread rum_angular/README.md Outdated
Comment thread rum_angular/CHANGELOG.md
@cit-pr-commenter-54b7da
Copy link
Copy Markdown

Monitor Template Quality Assessment

52 monitors analyzed across 9 integrations.

  • 34 monitors have missing sections
  • Most common missing sections: WHY, IMPACT, HOW_TO_TROUBLESHOOT
Monitors with missing sections
Integration Monitor Missing Sections Suggested Links
kubernetes [Kubernetes] Monitor Kubernetes Deployments Replica Pods IMPACT, WHY, HOW_TO_TROUBLESHOOT, RELATED_LINKS - Logs

@rgaignault
Copy link
Copy Markdown
Contributor Author

/merge

@gh-worker-devflow-routing-ef8351
Copy link
Copy Markdown

gh-worker-devflow-routing-ef8351 Bot commented Apr 22, 2026

View all feedbacks in Devflow UI.

2026-04-22 12:06:32 UTC ℹ️ Start processing command /merge


2026-04-22 12:06:39 UTC ℹ️ MergeQueue: waiting for PR to be ready

This pull request is not mergeable according to GitHub. Common reasons include pending required checks, missing approvals, or merge conflicts — but it could also be blocked by other repository rules or settings.
It will be added to the queue as soon as checks pass and/or get approvals. View in MergeQueue UI.
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.


2026-04-22 12:07:08 UTC ⚠️ MergeQueue: This merge request was unqueued

roman.gaignault@datadoghq.com unqueued this merge request

@rgaignault
Copy link
Copy Markdown
Contributor Author

/merge -c

@gh-worker-devflow-routing-ef8351
Copy link
Copy Markdown

gh-worker-devflow-routing-ef8351 Bot commented Apr 22, 2026

View all feedbacks in Devflow UI.

2026-04-22 12:07:02 UTC ℹ️ Start processing command /merge -c

@steveny91 steveny91 enabled auto-merge April 22, 2026 12:17
@steveny91 steveny91 disabled auto-merge April 22, 2026 12:29
@steveny91 steveny91 merged commit 425cb11 into master Apr 22, 2026
36 of 37 checks passed
@steveny91 steveny91 deleted the romanG/angular-integration-docs branch April 22, 2026 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

editorial review Waiting on a more in-depth review from a docs team editor mergequeue-status: removed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants