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 #11

Merged
merged 3 commits into from
Dec 4, 2024
Merged

Feature/hgi 6873 #11

merged 3 commits into from
Dec 4, 2024

Conversation

butkeraites-hotglue
Copy link
Contributor

No description provided.

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.
This commit enhances the discover_streams method in the TapKlaviyo class by adding error handling during the stream discovery process. Instead of failing silently, the method now logs an error message if an exception occurs while instantiating a stream class. This change improves the robustness of the integration by ensuring that issues with specific streams are reported, facilitating easier debugging and maintenance.
@butkeraites-hotglue butkeraites-hotglue self-assigned this Dec 2, 2024
@hsyyid hsyyid merged commit 48a23a0 into main Dec 4, 2024
0 of 4 checks passed
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