-
Notifications
You must be signed in to change notification settings - Fork 48
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
TDL-26703: Refactor entire tap #13
Draft
prijendev
wants to merge
89
commits into
envoy:master
Choose a base branch
from
singer-io:TDL-26703-tap-refactoring
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…mments) added, changes in bookmark_date updation
Add Credit Notes Stream
* remove max length constraint * bump to 0.0.5 * update base stream * update child object streams * bump to 0.0.6 * update metadata and stream properties * dump tap-framework dependency
…added promotional credits
* new streams(coupon_sets, coupon_codes, orders, gifts, creditnotes, comments) added, changes in bookmark_date updation * Continuing to other streams incase order stream is not enabled * removed comments, coupon_sets, coupon_codes * Added Orders, Gifts, Creditnotes in ReadMe * removed SELECTED=true, added REPLICATION_METHOD, REPLICATION_KEY and added promotional credits * reverted replication method to INCREMENTAL * promotional credit bookmark key changes * gift schema changes, order schema changes * Hard delete handling by reading events * removed selected-by-default
* added more objects in events content * added coupon sets and codes * Clean up whitespace in events.json
* TDL-6342: add additional fields from API and make invoices and promotional_credit common stream * Added fields in item_families and subscriptions stream * Updated start_date case to pass integration test * Deleted promotional_credits from item_model as it moves to common
* TDL-13904: Upgraded event stream schema. * TDL-13904: Restructured events.json stream * TDL-13904: Updated events stream schema * TDL-13904: Removed maximum and minimum * TDL-13904: Added coments and doc string * TDL-13904: comments grammer updated Co-authored-by: dbshah1212 <[email protected]> Co-authored-by: savan-chovatiya <[email protected]> Co-authored-by: Kyle Allan <[email protected]>
* TDL-6173: Updated Bookmark handling, date without tz will updated in UTC tz formate. * TDL-6173: Wrong format start date isn't supported in stitch hence removed that test * TDL-6173: Revert back the bookmarking logic changed by chargebee team * TDL-6173: Added start date format check * TDL-6173: Added Unittest to test wrong formate of start date * TDL-6173: Updated readme file * TDL-6173: Unit test updated * TDL-6173: Verify the start_date format in discover mode as well. * TDL-6173: Updated Unittest Co-authored-by: dbshah1212 <[email protected]> Co-authored-by: savan-chovatiya <[email protected]>
* Add MANIFEST.in * Bump to v1.2.1, update changelog Co-authored-by: Andy Lu <[email protected]>
* Update schema glob in MANIFEST.in * Bump to v1.2.2, update changelog
* TDL-6624: Added comments stream * TDL-6624: Changed record comparison in start date test. * TDL-6624: Taken lates test case changes * TDL-6624: Updated json of test * TDL-6624: Updated schema of comment stream * response to feedback * Add newline to file Co-authored-by: dbshah1212 <[email protected]> Co-authored-by: savan-chovatiya <[email protected]> Co-authored-by: Zach Harris <[email protected]>
* TDL-13631: Added include_deleted configuration * TDL-13631: Added Check for include_deleted * TDL-13631: Updated the test name * TDL-13631: Updated test name * TDL-13631: CHange in if condition * Fix tests * Fix include_delete test * TDL-13631: Updated test so it can run on both versions * TDL-13631: Upgraded test case. * TDL-13631: Updated integration test * TDL-13631: Updated test * TDL-13631: Updated test. * TDL-13631: Enhnaced test Co-authored-by: dbshah1212 <[email protected]> Co-authored-by: savan-chovatiya <[email protected]> Co-authored-by: Andy Lu <[email protected]> Co-authored-by: zachharris1 <[email protected]>
…from API doc. (#69) * Added missing additional fields * Added missing field in events schema * Added remaining fields for item_prices stream * Added missing fields which are not present in doc but found in testing * Added missing fields for event's content and subscription * Added extra type string for items and item_prices * Updated customer and subscription schema * Updated invoice schema Co-authored-by: zachharris1 <[email protected]>
* error handling for product catalog version API * added API response log * GEN-763 Added Quotes Object for Syncing to Stitch * case insensitive comparison of site name * Missed comma Co-authored-by: cb-akashpandey <[email protected]> Co-authored-by: cb-prasanna <[email protected]>
* bump 1.3.1 * add quotes to tests * skip quotes stream in test
* Added calls to generate events * Update test
* reverted bookmark logic * updated the code to take minimum of max replication key and now - 2 min as bookmark * added unittests
* TDL-19099: Add missing tap-tester tests (#83) * inital commit: add missing tap tester tests * updated the code to get event_type for events stream before transform * updated integration tests * resolve review comments * resolve integration test failure * added all fields comments * updated tap tester tests * removed pylint disable statement * updated config.yml file * fixed event_type error * updated comment * addressed review comments * updated config.yml file and start date test * TDL-16315: Implement request timeout (#78) * Added request timeout * Updated config.yml * Updated readme file * Added backoff for connectionError * updated test cases Co-authored-by: harshpatel4crest <[email protected]> * TDL-19101: Add custom exception handling (#85) * Added custom exception handling * Added unit tests * Updated config.yml to cover unit test report * Added condition to handle status code other than 4xx 5xx * Added condition to handle status code other than 4xx 5xx * Added/updated unit tests * Added existing unit tests * Resolved review comment * Fixed unit test * Resolved review comment * Removed redundent Server429Error Co-authored-by: harshpatel4crest <[email protected]> * TDL-19489: Revert back bookmark logic (#88) * TDL-16367: Fix pagination test failure (#79) * Added calls to generate events * Update test * TDL-19270: Revert back bookmarking logic (#86) * reverted bookmark logic * updated the code to take minimum of max replication key and now - 2 min as bookmark * added unittests * Bump version * reverted bookmark logic Co-authored-by: savan-chovatiya <[email protected]> Co-authored-by: KrishnanG <[email protected]> * TDL-19102: Add missing fields to schema (#87) * inital commit: add missing tap tester tests * updated the code to get event_type for events stream before transform * updated integration tests * resolve review comments * resolve integration test failure * added all fields comments * added missing and new fields in the schema * updated tap tester tests * removed pylint disable statement * updated config.yml file * fixed event_type error * updated comment * addressed review comments * updated config.yml file and start date test * updated test cases as per missing fields * updated bookmark test * resolved bookmark test failure Co-authored-by: savan-chovatiya <[email protected]> Co-authored-by: KrishnanG <[email protected]>
* custom fields for PC2.0 (#92) * Modified the all fields test (#93) Co-authored-by: cb-nandita <[email protected]> Co-authored-by: Umang Agrawal <[email protected]>
Co-authored-by: RushiT0122 <[email protected]>
* add missing customer details * update integration tests --------- Co-authored-by: RushiT0122 <[email protected]>
tax_rate can't be stored as an integer because many states allow fractional tax rates This change makes it match `credit_notes.json` (which stores correctly as a float) Co-authored-by: Wynn Slater <[email protected]>
Co-authored-by: RushiT0122 <[email protected]>
* Update subscription schema to include manual discounts. * Fixed All fields integration tests. * Fixed All fields integration tests. * Fixed all fields & pagination tests. * Fixed All fields integration test. * minor bump 1.4.0 (#110)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.