-
Notifications
You must be signed in to change notification settings - Fork 535
pipeline: outputs: datadog: added documentation for site configuration field #2032
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
base: master
Are you sure you want to change the base?
pipeline: outputs: datadog: added documentation for site configuration field #2032
Conversation
Signed-off-by: Lucas Tembras <lucas.tembras@datadoghq.com>
64bf6ba
to
ec88270
Compare
Signed-off-by: Lucas Tembras <lucas.tembras@datadoghq.com>
Signed-off-by: Lucas Tembras <51347268+lucastemb@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for submitting a PR @lucastemb 😁 I made one small suggestion to match the phrasing we've used in other pages, but otherwise LGTM!
one question, though: is the feature itself still unmerged? if so, you can add the waiting-on-code-merge
label to this docs PR to make sure none of us merge it prematurely.
Co-authored-by: Alexa Kreizinger <alexakreizinger@gmail.com> Signed-off-by: Lucas Tembras <51347268+lucastemb@users.noreply.github.com>
@alexakreizinger Yup, still waiting for approval on the feature, but I don't believe I have the option to add the label myself. 😢 |
I executed CI workflows and added a suggested label. Thanks for your work! 👍 |
Added supporting documentation for the following PR that seeks to add support for a
site
configuration field to give customers options on where to send telemetry data.