Skip to content

✨ RUM Browser NextJS Integration#2918

Merged
steveny91 merged 10 commits intomasterfrom
beltran.bulbarella/nextjs-approuter-integration
Apr 21, 2026
Merged

✨ RUM Browser NextJS Integration#2918
steveny91 merged 10 commits intomasterfrom
beltran.bulbarella/nextjs-approuter-integration

Conversation

@BeltranBulbarellaDD
Copy link
Copy Markdown
Contributor

@BeltranBulbarellaDD BeltranBulbarellaDD commented Mar 4, 2026

What does this PR do?

Now that we have the Browser-SDK NextJS package as private we want to make the Documentation page for it.

Motivation

What inspired you to submit this pull request?
This RFC

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
  • If this PR includes a log pipeline, please add a description describing the remappers and processors.

Additional Notes

Anything else we should know when reviewing?

@BeltranBulbarellaDD BeltranBulbarellaDD force-pushed the beltran.bulbarella/nextjs-approuter-integration branch from 3a4204c to 5695cf0 Compare March 16, 2026 13:50
@BeltranBulbarellaDD BeltranBulbarellaDD marked this pull request as ready for review March 24, 2026 11:10
@BeltranBulbarellaDD BeltranBulbarellaDD requested review from a team as code owners March 24, 2026 11:10
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0505628cdb

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread rum_nextjs/README.md Outdated
Comment thread rum_nextjs/README.md Outdated
@BeltranBulbarellaDD
Copy link
Copy Markdown
Contributor Author

@codex pls review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3b6f9ec3e0

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread rum_nextjs/README.md Outdated
@BeltranBulbarellaDD
Copy link
Copy Markdown
Contributor Author

@codex pls review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Can't wait for the next one!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@BeltranBulbarellaDD
Copy link
Copy Markdown
Contributor Author

@codex pls review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Swish!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@BeltranBulbarellaDD BeltranBulbarellaDD marked this pull request as draft March 24, 2026 14:37
@BeltranBulbarellaDD BeltranBulbarellaDD marked this pull request as ready for review April 7, 2026 16:03
@joepeeples joepeeples added the editorial review Waiting on a more in-depth review from a docs team editor label Apr 7, 2026
@joepeeples
Copy link
Copy Markdown

joepeeples commented Apr 7, 2026

Opened DOCS-13925 to assign a Docs writer and follow up with editorial review.

Copy link
Copy Markdown
Contributor

@rtrieu rtrieu left a comment

Choose a reason for hiding this comment

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

some feedback for your review. let me know if you have any questions about my feedback or want to discuss. thanks for your PR!

Comment thread rum_nextjs/manifest.json Outdated
Comment thread rum_nextjs/manifest.json
"Category::Metrics",
"Category::Network",
"Category::Tracing",
"Supported OS::Android",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

should this be here, since it runs in browsers vs mobile OS

Comment thread rum_nextjs/manifest.json
Comment thread rum_nextjs/manifest.json
Comment thread rum_nextjs/manifest.json Outdated
Comment thread rum_nextjs/README.md Outdated
Comment thread rum_nextjs/README.md Outdated
Comment thread rum_nextjs/README.md
Comment thread rum_nextjs/README.md
Comment thread rum_nextjs/README.md Outdated
@BeltranBulbarellaDD
Copy link
Copy Markdown
Contributor Author

BeltranBulbarellaDD commented Apr 20, 2026

@rtrieu thanks for the comments! I've addressed them all except the ones in manifest.json. I've based myself mostly on what Rum-React does Link, so I don't think we need to add assets or remove the Mobile part since React has it.

@cit-pr-commenter-54b7da
Copy link
Copy Markdown

cit-pr-commenter-54b7da Bot commented Apr 20, 2026

Monitor Template Quality Assessment

52 monitors analyzed across 9 integrations.

  • 31 monitors have missing sections
  • Most common missing sections: WHY, HOW_TO_TROUBLESHOOT, IMPACT
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

steveny91
steveny91 previously approved these changes Apr 21, 2026
@temporal-github-worker-1 temporal-github-worker-1 Bot dismissed steveny91’s stale review April 21, 2026 16:23

Review from steveny91 is dismissed. Related teams and files:

  • agent-integrations
    • .github/CODEOWNERS
@steveny91 steveny91 added this pull request to the merge queue Apr 21, 2026
Merged via the queue into master with commit 779c8e2 Apr 21, 2026
36 checks passed
@steveny91 steveny91 deleted the beltran.bulbarella/nextjs-approuter-integration branch April 21, 2026 19:47
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants