Skip to content

Conversation

@lhelman
Copy link

@lhelman lhelman commented Jan 13, 2026

What does this PR do?

A brief description of the change being made with this pull request.

Motivation

What inspired you to submit this pull request?

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?

@lhelman lhelman requested review from a team as code owners January 13, 2026 14:55
@dd-dominic dd-dominic changed the title feat: auth0 added auth0.api_request.count metric [ECOINT-353] feat: auth0 added auth0.api_request.count metric Jan 13, 2026
@buraizu buraizu added the editorial review Waiting on a more in-depth review from a docs team editor label Jan 13, 2026
@buraizu
Copy link
Contributor

buraizu commented Jan 13, 2026

Created DOCS-13104 for documentation team review

lhelman and others added 13 commits January 15, 2026 09:52
Co-authored-by: Dominic Medina <115744456+dd-dominic@users.noreply.github.com>
Co-authored-by: Dominic Medina <115744456+dd-dominic@users.noreply.github.com>
Co-authored-by: Dominic Medina <115744456+dd-dominic@users.noreply.github.com>
Co-authored-by: Dominic Medina <115744456+dd-dominic@users.noreply.github.com>
Co-authored-by: Dominic Medina <115744456+dd-dominic@users.noreply.github.com>
Co-authored-by: Dominic Medina <115744456+dd-dominic@users.noreply.github.com>
Co-authored-by: Dominic Medina <115744456+dd-dominic@users.noreply.github.com>
Co-authored-by: Dominic Medina <115744456+dd-dominic@users.noreply.github.com>
Co-authored-by: Dominic Medina <115744456+dd-dominic@users.noreply.github.com>
Co-authored-by: Dominic Medina <115744456+dd-dominic@users.noreply.github.com>
Co-authored-by: Dominic Medina <115744456+dd-dominic@users.noreply.github.com>
Co-authored-by: Dominic Medina <115744456+dd-dominic@users.noreply.github.com>
@lhelman
Copy link
Author

lhelman commented Jan 21, 2026

@dd-dominic I didn't really change anything on the grok for the logs, do I need to fix that now? If I need to fix it, Is there any way to obtain more information about that error?

Which error? I don't see any failed tests at the moment.

It didn't run yet (it says "queued"), It finally run succesfully,
a previous example of failed run was:
https://github.com/DataDog/integrations-extras/pull/2879/checks?check_run_id=60792210936

@lhelman
Copy link
Author

lhelman commented Jan 22, 2026

Is there anything else from my side?

@lhelman lhelman requested a review from dd-dominic January 22, 2026 19:23
Copy link
Collaborator

@dd-dominic dd-dominic left a comment

Choose a reason for hiding this comment

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

Replace the individual widget screenshots with 1 expanded screenshot. See attached example.
Screenshot 2026-01-22 at 5 19 39 PM

@lhelman lhelman requested a review from dd-dominic January 23, 2026 13:56
@dd-dominic
Copy link
Collaborator

Waiting on Docs Team to take a look, then we can merge

Copy link
Contributor

@evazorro evazorro left a comment

Choose a reason for hiding this comment

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

I left some wording suggestions for clarity and style. Thank you for updating the documentation!

lhelman and others added 6 commits February 6, 2026 15:44
Co-authored-by: Eva Parish <eva.parish@datadoghq.com>
Co-authored-by: Eva Parish <eva.parish@datadoghq.com>
Co-authored-by: Eva Parish <eva.parish@datadoghq.com>
Co-authored-by: Eva Parish <eva.parish@datadoghq.com>
Co-authored-by: Eva Parish <eva.parish@datadoghq.com>
@lhelman lhelman requested a review from evazorro February 6, 2026 18:45
@dd-dominic dd-dominic requested review from a team and davidfeng-datadog and removed request for a team February 6, 2026 20:06
@davidfeng-datadog
Copy link
Contributor

PR looks good on staging. Will create a PR to add auth0. to the standard metric allowlist, merge that, then merge this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants