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

Added _id and _cid to the schema for generated streams #14647

Merged
merged 1 commit into from
Jul 13, 2022

Conversation

alexander-marquardt
Copy link
Contributor

What

This connector automatically populates the json schema, however the schema that is pulled back from Webflow does not include the _id (record id) or the _cid (collection id) fields in the records that are returned. These are now manually added in after pulling the webflow schema back to Airbyte.

How

These two fields are added into the dictionary returned by the get_json_schema method inside CollectionContents

I have confirmed on my local version of Airbyte that these fields now appear in the catalog, as demonstrated in the image below:

image

@github-actions github-actions bot added the area/connectors Connector related issues label Jul 12, 2022
@alexander-marquardt alexander-marquardt changed the title Added _id and _cid to the schema (catalog) Added _id and _cid to the schema for generated streams which correspond to Webflow collections Jul 12, 2022
@alexander-marquardt alexander-marquardt changed the title Added _id and _cid to the schema for generated streams which correspond to Webflow collections Added _id and _cid to the schema for generated streams Jul 12, 2022
@marcosmarxm marcosmarxm removed their assignment Jul 13, 2022
@alexander-marquardt alexander-marquardt requested a review from a team July 13, 2022 16:28
@alexander-marquardt alexander-marquardt merged commit f412716 into master Jul 13, 2022
@alexander-marquardt alexander-marquardt deleted the webflow_add_ids_to_streams branch July 13, 2022 19:15
@sherifnada
Copy link
Contributor

@alexander-marquardt shouldn't we publish a new version of the connector as part of this?

@alexander-marquardt
Copy link
Contributor Author

alexander-marquardt commented Jul 13, 2022

Oh, probably - what are the steps for that? I assumed that it would automagically happen ..

@alexander-marquardt
Copy link
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants