search

airbytehq / airbyte

The leading data integration platform for ETL / ELT data pipelines from APIs, databases & files to data warehouses, data lakes & data lakehouses. Both self-hosted and Cloud-hosted.
https://airbyte.com
Other
16.07k stars 4.11k forks source link

:sparkles: Source Todoist : Migrate Python CDK to Low-code CDK #30878

Open bmcgilli opened 1 year ago

bmcgilli commented 1 year ago

https://docs.airbyte.com/integrations/sources/todoist

Rules for the Connectors Hackathon You can only work on one connector at a time Make sure to resolve all suggestions from the Airbyte team Must include all streams of the connector you’re migrating. Otherwise, it will not be accepted. If you don’t commit anything within five days, the issue may be assigned to someone else.

shivam096 commented 1 year ago

Hey @bmcgilli ,

Can I pick this up ?

justinbchau commented 1 year ago

all yours @shivam096

shivam096 commented 1 year ago

Hey @justinbchau ,

I have started to work on this. Apologies for the delay in response

justinbchau commented 1 year ago

Per the rules of the Hackathon, I must open this issue to everyone if there has been no commits in 5 days. Are you still working on this @shivam096

topefolorunso commented 1 year ago

I want to work on this @justinbchau if @shivam096 is not available.

shivam096 commented 1 year ago

Hey Justin,

Yes I have almost completed the implementation and only testing is left

justinbchau commented 1 year ago

Will you be pushing a PR for this soon @shivam096 ? Today is day 5 which is the cutoff for issues with no PRs

topefolorunso commented 1 year ago

@justinbchau You can assign this issue to me next if no response from @shivam096

shivam096 commented 1 year ago

Hey Tope,

I have completed the implementation and would be raising a PR tomorrow as I need to update the tests a bit. Also Justin apologies for the delay, I'm was a bit preoccupied with my assignments but will do this tomorrow for sure

shivam096 commented 1 year ago

Hey @justinbchau ,

Raised the PR - #31903

justinbchau commented 1 year ago

Can you please link the PR to this issue?

shivam096 commented 1 year ago

Hey @justinbchau ,

I dont think I have access to do it. Here is the PR I raised - https://github.com/airbytehq/airbyte/pull/31903

shivam096 commented 11 months ago

Hey @justinbchau

I was asked to open a new PRR for this issue since the previous one had merge conflicts due to airbyte's testing changes. So I have raised a new PR - #32690 (https://github.com/airbytehq/airbyte/pull/32690).

shivam096 commented 10 months ago

Hey @justinbchau ,

The PR is merged :)