-
Notifications
You must be signed in to change notification settings - Fork 30
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
cookiecutter write_schema(): JSON not serializable #13
Comments
Since it seems Singer Slack channel is free (thus has message limits and will make history not accessible), I'm copy-pasting the message with the workaround solution: Author: Steven Ram
Thanks also to Rebecca Sanjabi for linking to Steven's original message. I tested and the solution works. |
Awesome guys! I was running into the same problem, and your solution works for me, too. Thanks! |
I had the same problem that I managed to trace back further upstream. It seems like the catalog command line argument isn't getting passed so that sync mode is falling back to calling discover instead of reading the catalog.
It seems discover() is returning the Schema class which is not JSON Serializable. Here's the traceback I get when running the following command: ~/.virtualenvs/tap-woocommerce/bin/tap-woocommerce --config tap-woocommerce/config.json --catalog tap-woocommerce/catalog.json | ~/.virtualenvs/target-csv/bin/target-csv
The fixes proposed earlier work, but hopefully this helps solve the root problem. |
#17 fixes the issue |
When following documentation for setting up tap-template, user in Slack first reported issue with executing
singer.write_schema().
Have to call to dict to_dict() on schema. See Slack message:https://singer-io.slack.com/archives/C2TGFCZEV/p1584891258008800
The text was updated successfully, but these errors were encountered: