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

partitions = None , workaround #119

Open
visch opened this issue Apr 24, 2022 · 0 comments
Open

partitions = None , workaround #119

visch opened this issue Apr 24, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@visch
Copy link
Contributor

visch commented Apr 24, 2022

This exists in every stream that has a parent stream, but doesn't need partitions set.

 # TODO not clear why this is needed
 partitions = None

This is harmless but it is distracting and we should dive into this to fix it. Basically what's happening is due to us playing with an internal sdk function https://github.com/AutoIDM/tap-clickup/pull/117/files#diff-4490b7dbe49a82fea4fdf63d08c6c503536d30f56841c428f2998bf03b4b94a2L141 we have partitions being set in places that we don't want them to be set.

@visch visch added the enhancement New feature or request label Mar 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant