-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
🐛 Source Quickbooks: use number data type for decimal fields #4986
🐛 Source Quickbooks: use number data type for decimal fields #4986
Conversation
/test connector=source-quickbooks-singer
|
/test connector=source-quickbooks-singer
|
/test connector=source-quickbooks-singer
|
/test connector=source-quickbooks-singer
|
/test connector=source-quickbooks-singer
|
/test connector=source-quickbooks-singer
|
/test connector=source-quickbooks-singer
|
/test connector=source-quickbooks-singer
|
dependencies { | ||
implementation files(project(':airbyte-integrations:bases:base-standard-source-test-file').airbyteDocker.outputs) | ||
implementation files(project(':airbyte-integrations:bases:source-acceptance-test').airbyteDocker.outputs) | ||
implementation files(project(':airbyte-integrations:bases:base-singer').airbyteDocker.outputs) |
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.
don't need docker image for singer anymore, we are using cdk PYPI package, which already includes singer wrappers
implementation files(project(':airbyte-integrations:bases:base-singer').airbyteDocker.outputs) |
empty_streams: ["budgets", "classes", "departments", "payment_methods", "transfers", "vendor_credits"] | ||
# incremental: | ||
# This test is commented out, because singer tap expects LastUpdatedTime for each stream in state. | ||
# But records contain field MetaData with nested LastUpdatedTime. It causes test error. |
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.
not true, SAT supports absolute paths
@@ -0,0 +1,7 @@ | |||
#!/usr/bin/env sh |
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.
could you update it in accordance to latest file from the templates?
/test connector=source-quickbooks-singer
|
/test connector=source-quickbooks-singer
|
/test connector=source-quickbooks-singer
|
/publish connector=connectors/source-quickbooks-singer
|
What
Using number data type for decimal fields instead string
How
Created fork of singer tap, changed schemes. Also migrated Source to Airbyte CDK.
Closes #4292
Pre-merge Checklist
Expand the checklist which is relevant for this PR.
Connector checklist
airbyte_secret
in the connector's spec./gradlew :airbyte-integrations:connectors:<name>:integrationTest
./test connector=connectors/<name>
command as documented here is passing.README.md
docs/SUMMARY.md
if it's a new connectordocs/integrations/<source or destination>/<name>
.docs/integrations/...
. See changelog exampledocs/integrations/README.md
contains a reference to the new connector/publish
command described here