AutoIDM / tap-clickup

tap-clickup , singer compliant tap for pulling clickup data
MIT License
12 stars 20 forks source link

Fixed url params for FolderlessTaskStream and FolderTaskStream #80

Closed LucasZielke closed 3 years ago

LucasZielke commented 3 years ago

For whatever reason the clickup API does not accept lowercase true specifically in archived url param, so to maintain consistency just title cased all three

visch commented 3 years ago

This is odd, I verified that data was being pulled with true.

Just to clarify tasks are certainly flowing right now with archived being set to true , not True.

I tested on a different production instance, and recieved a bunch more Tasks with archived set to true. Verifying the count difference right now

visch commented 3 years ago

Found it.

If archived=true, you will not get any tasks that are not archived.

visch commented 3 years ago

Fixed in #81