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

Feature/hgi 6873 #8

Merged
merged 2 commits into from
Dec 2, 2024
Merged

Feature/hgi 6873 #8

merged 2 commits into from
Dec 2, 2024

Conversation

butkeraites-hotglue
Copy link
Contributor

No description provided.

@butkeraites-hotglue butkeraites-hotglue self-assigned this Nov 27, 2024
@butkeraites-hotglue butkeraites-hotglue marked this pull request as ready for review December 2, 2024 15:39
This commit introduces a new stream, ReviewsStream, to the Klaviyo integration. The ReviewsStream is defined with its own name, path, primary keys, and replication key, allowing for the extraction of review data from the Klaviyo API. This enhances the data extraction capabilities of the tap.
- Updated the HTTP headers revision date in KlaviyoStream to "2024-10-15".
- Introduced methods for loading schemas and filling missing properties in KlaviyoStream, improving schema handling.
- Added a new ReviewsStream with updated replication key and URL parameterization for pagination.
- Created a new JSON schema file for reviews, defining the structure and required properties for review data.

These changes improve the functionality and robustness of the Klaviyo integration, enabling better data extraction and schema management.
def load_schema(self, name):
return json.load(open(self.get_abs_path('schemas/{}.json'.format(name))))

def _fill_missing_properties(self, property_list):
Copy link
Member

Choose a reason for hiding this comment

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

What does this function do?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It gets from a schema file a list of default properties to complement the dynamic discover

@hsyyid hsyyid merged commit 4cbfd38 into main Dec 2, 2024
0 of 4 checks passed
@hsyyid hsyyid deleted the feature/hgi-6873 branch December 2, 2024 21:34
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