Skip to content
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

feat: documenting the new privacy flag #37

Merged
merged 2 commits into from
Aug 12, 2024

Conversation

deborahgu
Copy link
Member

Per sensible request, adding documentation about how to use the new privacy changing flag for DD

Per sensible request, adding documentation about how to use the new
privacy changing flag for DD
Copy link
Member

@adamstankiewicz adamstankiewicz left a comment

Choose a reason for hiding this comment

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

Left a couple nit-level suggestions to improve the README further, but approved regardless.

README.md Outdated
@@ -31,6 +31,10 @@ In the `env.config.js` file, add the following

The DatadogLoggingService is a concrete implementation of the logging service interface that sends messages to Datadog that can be seen in Datadog RUM and Datadog browser logs. When in development mode, all messages will instead be sent to the console.

When you configure a Datadog logging implementation, you'll set a number of variables used by the DatadogLoggingService. Most (eg. `DATADOG_APPLICATION_ID`, `DATADOG_SESSION_SAMPLE_RATE`) are either supplied to you by Datadog when you set up a RUM application, or are options about how much data we should be saving. Optional variables include:
Copy link
Member

Choose a reason for hiding this comment

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

nit: Is it worth explicitly listing out all the possible environment variable config settings here, which are required/optional, etc. so consumers don't necessarily need to read the code implementation to understand how to configure the MFE?

Copy link
Member Author

Choose a reason for hiding this comment

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

I did that at first, and then I second-guessed myself, so I'll take your nit as a gut-check on that and go back to my original.

README.md Outdated
@@ -31,6 +31,10 @@ In the `env.config.js` file, add the following

The DatadogLoggingService is a concrete implementation of the logging service interface that sends messages to Datadog that can be seen in Datadog RUM and Datadog browser logs. When in development mode, all messages will instead be sent to the console.

When you configure a Datadog logging implementation, you'll set a number of variables used by the DatadogLoggingService. Most (eg. `DATADOG_APPLICATION_ID`, `DATADOG_SESSION_SAMPLE_RATE`) are either supplied to you by Datadog when you set up a RUM application, or are options about how much data we should be saving. Optional variables include:
Copy link
Member

Choose a reason for hiding this comment

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

nit: While the Datadog RUM application's SDK configuration provides values for DATADOG_SESSION_SAMPLE_RATE and other settings, it's not really something that needs to be supplied from Datadog (i.e., you can freely change the value as needed for your application), unlike the DATADOG_APPLICATION_ID and DATADOG_CLIENT_TOKEN config settings which are specific Datadog-provided values.

Adding all possible DD settings.
@deborahgu deborahgu merged commit 6a7c530 into master Aug 12, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants