Skip to content

Update data-governance.mdx #574

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update data-governance.mdx #574

wants to merge 2 commits into from

Conversation

aisz0811
Copy link
Contributor

Describe the purpose of your pull request

  • Provide a clear and concise description of the changes.

Related issues (only if applicable)

  • Provide links to issues relating to this pull request.

How to test? (only if applicable)

  • What part of the application was affected by the changes? What should be tested?

Requirement checklist

  • I have validated my changes on a test/local environment.
  • I have tested that the code snippets I added work. (Leave unchecked if there are no new code snippets.)
  • I have added my changes to the V1 and V2 documentations.

@aisz0811 aisz0811 requested a review from a team as a code owner June 30, 2025 12:16
Copy link

cloudflare-workers-and-pages bot commented Jun 30, 2025

Deploying configcat-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 4e3ab65
Status: ✅  Deploy successful!
Preview URL: https://ef152042.docs-xmy.pages.dev
Branch Preview URL: https://aisz0811-patch-6.docs-xmy.pages.dev

View logs

@@ -44,10 +42,10 @@ in your application code when it is being initialized.

## Troubleshooting

#### What if I forgot to pass the `dataGovernance` parameter?
#### What happens if I forget to pass the `dataGovernance` parameter?

The ConfigCat backend will take the Dashboard preference as primary. Having `EU Only` selected on the Dashboard but forgot to pass the proper `dataGovernance` parameter to the SDK. In this case your config JSONs will only published to the EU CDN nodes. And every config JSON download made from your application to any node outside the EU, the request will be redirected automatically to the EU CDN. Recommended to deploy a fix with the correct `dataGovernance` param, since response times will be significantly longer.
Copy link
Contributor

@adams85 adams85 Jun 30, 2025

Choose a reason for hiding this comment

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

This section could be improved too as it's pretty incoherent and hard to follow.

@@ -44,10 +42,10 @@ in your application code when it is being initialized.

## Troubleshooting

#### What if I forgot to pass the `dataGovernance` parameter?
#### What happens if I forget to pass the `dataGovernance` parameter?

The ConfigCat backend will take the Dashboard preference as primary. Having `EU Only` selected on the Dashboard but forgot to pass the proper `dataGovernance` parameter to the SDK. In this case your config JSONs will only published to the EU CDN nodes. And every config JSON download made from your application to any node outside the EU, the request will be redirected automatically to the EU CDN. Recommended to deploy a fix with the correct `dataGovernance` param, since response times will be significantly longer.

#### `Warning: Your dataGovernance parameter at ConfigCatClient initialization is not in sync with your preferences...`
Copy link
Contributor

@adams85 adams85 Jun 30, 2025

Choose a reason for hiding this comment

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

This warning message has changed, see also https://docs.google.com/spreadsheets/d/1twC9G40PQK7cmUXTAVEgNMvDdFJxoN9W6WTLOU9wRM4/edit?usp=sharing

Suggested change
#### `Warning: Your dataGovernance parameter at ConfigCatClient initialization is not in sync with your preferences...`
#### `Warning: The dataGovernance parameter specified at the client initialization is not in sync with the preferences on the ConfigCat Dashboard....`

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.

2 participants