-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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 low-code documentation #15748
Conversation
@@ -135,7 +135,7 @@ In this example, outer.inner.k2 will evaluate to "MyKey is MyValue" | |||
## References | |||
|
|||
Strings can contain references to previously defined values. | |||
The parser will dereference these values to produce a complete ConnectionDefinition | |||
The parser will dereference these values to produce a complete object definition. |
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.
ConnectionDefinition
is not a term we define anywhere
@@ -3,7 +3,7 @@ | |||
## From scratch | |||
|
|||
- [Overview](overview.md) | |||
- [Yaml structure](overview.md) | |||
- [Yaml structure](yaml-structure.md) |
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.
wrong link
@@ -51,12 +51,12 @@ A stream generally corresponds to a resource within the API. They are analogous | |||
A stream is defined by: | |||
|
|||
1. A name | |||
2. Primary key (Optional): Used to uniquely identify records, enabling deduplication. Can be a string for single primary keys, a list of strings for composite primary keys, or a list of list of strings for composite primary keys consisting of nested fields. | |||
2. Primary key (Optional): Used to uniquely identify records, enabling deduplication. Can be a string for single primary keys, a list of strings for composite primary keys, or a list of list of strings for composite primary keys consisting of nested fields |
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.
remove dots for consistency
More details on the record selector can be found in the [record selector section](record-selector.md) | ||
More details on the stream slicers can be found in the [stream slicers section](stream-slicers.md) | ||
More details on the paginator can be found in the [pagination section](pagination.md) | ||
More details on the record selector can be found in the [record selector section](record-selector.md). |
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.
add newlines because these all show up as a single line in the docs
@@ -164,7 +163,7 @@ retriever: | |||
stream_slice_field: "repository" | |||
``` | |||
|
|||
[^1] This is a slight oversimplification. See update cursor section for more details on how the cursor is updated | |||
[^1] This is a slight oversimplification. See [update cursor section](#cursor-update) for more details on how the cursor is updated. |
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.
add a link to the section
@@ -8,7 +8,7 @@ Throughout this tutorial, we'll walk you through the creation an Airbyte source | |||
|
|||
We'll build a connector reading data from the Exchange Rates API, but the steps will apply to other HTTP APIs you might be interested in integrating with. | |||
|
|||
The API documentations can be found [here](https://exchangeratesapi.io/documentation/). | |||
The API documentations can be found [here](https://apilayer.com/marketplace/exchangerates_data-api). |
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.
point to their updated api docs
@@ -141,7 +141,7 @@ definitions: | |||
base: "{{ config['base'] }}" | |||
``` | |||
|
|||
The full connection definition should now look like | |||
The full connector definition should now look like |
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.
this is a connector, not a connection
* newlines * remove dots for consistency * a few more edits * more edits
What
A few more edits/corrections to the low-code documentation.
How
Recommended reading order
x.java
y.python
🚨 User Impact 🚨
Are there any breaking changes? What is the end result perceived by the user? If yes, please merge this PR with the 🚨🚨 emoji so changelog authors can further highlight this if needed.
Pre-merge Checklist
Expand the relevant checklist and delete the others.
New Connector
Community member or Airbyter
airbyte_secret
./gradlew :airbyte-integrations:connectors:<name>:integrationTest
.README.md
bootstrap.md
. See description and examplesdocs/integrations/<source or destination>/<name>.md
including changelog. See changelog exampledocs/integrations/README.md
airbyte-integrations/builds.md
Airbyter
If this is a community PR, the Airbyte engineer reviewing this PR is responsible for the below items.
/test connector=connectors/<name>
command is passing/publish
command described hereUpdating a connector
Community member or Airbyter
airbyte_secret
./gradlew :airbyte-integrations:connectors:<name>:integrationTest
.README.md
bootstrap.md
. See description and examplesdocs/integrations/<source or destination>/<name>.md
including changelog. See changelog exampleAirbyter
If this is a community PR, the Airbyte engineer reviewing this PR is responsible for the below items.
/test connector=connectors/<name>
command is passing/publish
command described hereConnector Generator
-scaffold
in their name) have been updated with the latest scaffold by running./gradlew :airbyte-integrations:connector-templates:generator:testScaffoldTemplates
then checking in your changesTests
Unit
Put your unit tests output here.
Integration
Put your integration tests output here.
Acceptance
Put your acceptance tests output here.