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

🪟 🎨 Update text describing connection geography #19879

Merged
merged 2 commits into from
Dec 1, 2022

Conversation

josephkmh
Copy link
Contributor

What

Updates the text describing what a connection's geography setting means. This was a suggestion from @alex-gron that @andyjih approved ✅

Before:

Depending on your network configuration, you may need to add IP addresses to your allowlist.

After:

Choose where the data for this connection will be processed. Depending on your network configuration, you may need to add IP addresses to your allowlist. Learn more.

This appears on both the connection creation page and on the settings tab of existing connections:
image
image

@octavia-squidington-iv octavia-squidington-iv added area/platform issues related to the platform area/frontend Related to the Airbyte webapp labels Nov 29, 2022
@@ -29,8 +29,10 @@ export const links = {
webhookVideoGuideLink: "https://www.youtube.com/watch?v=NjYm8F-KiFc",
webhookGuideLink: `${BASE_DOCS_LINK}/operator-guides/configuring-sync-notifications/`,
cronReferenceLink: "http://www.quartz-scheduler.org/documentation/quartz-2.3.0/tutorials/crontrigger.html",
cloudAllowlistIPsLink: `${BASE_DOCS_LINK}/cloud/getting-started-with-airbyte-cloud/#allowlist-ip-address`,
cloudAllowlistIPsLink: `${BASE_DOCS_LINK}/cloud/getting-started-with-airbyte-cloud/#allowlist-ip-addresses`,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I noticed the URL for the docs was changed as well - not 404-ing, but the link to the heading was broken 🙂

@josephkmh josephkmh marked this pull request as ready for review November 29, 2022 15:56
@josephkmh josephkmh requested a review from a team as a code owner November 29, 2022 15:56
@josephkmh josephkmh requested a review from timroes November 30, 2022 17:50
Copy link
Collaborator

@timroes timroes left a comment

Choose a reason for hiding this comment

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

Code LGTM, have not tested locally.

@josephkmh josephkmh merged commit 6a71f44 into master Dec 1, 2022
@josephkmh josephkmh deleted the joey/update-data-residency-descriptive-text branch December 1, 2022 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/frontend Related to the Airbyte webapp area/platform issues related to the platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants