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

move disambiguation section for connectors, connections, and configur… #39134

Merged
merged 1 commit into from
Jun 7, 2024

Conversation

Adorism
Copy link
Contributor

@Adorism Adorism commented Jun 5, 2024

Reorder concept introduction. There was a section to disambiguate between connectors, connections, and configured sources, but it was included in the Connection Builder Intro and seemed out of place. We should ensure people understand the relationship between these concepts before this. I moved it to the Connector Development Overview.

What

Logical sequence of introducing an in-product concept.

How

Moved to a more widely applicable overview document

Review guide

User Impact

Users should be primed to see these closely related concepts as they build new connectors.

Can this PR be safely reverted and rolled back?

  • YES 💚
  • NO ❌

Copy link

vercel bot commented Jun 5, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
airbyte-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 5, 2024 5:21pm

@octavia-squidington-iii octavia-squidington-iii added the area/documentation Improvements or additions to documentation label Jun 5, 2024
@Adorism Adorism merged commit 7cc188e into master Jun 7, 2024
27 checks passed
@Adorism Adorism deleted the audrey/connector-concepts-moved branch June 7, 2024 16:35
@erohmensing
Copy link
Contributor

Lol, somehow got here a minute after you merged. Thank you for this! ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants