-
Notifications
You must be signed in to change notification settings - Fork 7
New Source: Zoom #111
Comments
Hi @RealChrisSean, could you please share what data specifically we want to import from the |
Hi @RealChrisSean ,could you please provide the description? |
@misteryeo Can you help here? I can't access the original asana task. The best I have right now is this: airbytehq/airbyte#3696. |
Hello everyone! Let me try to bring a little bit of clarity and direction here. We would ideally want to have parity with the docs described here. Does this make sense? This connector was originally Singer-based but now, we want to move it to our low-code framework and it seems easier to write it from scratch than migrate it. Original code can be found here for reference. |
@RealChrisSean may I take this issue? It will help me gain better insights into the low-code cdk. I do have another issue assigned, but we're still missing some clarity with the approach. Let me take this one meanwhile? |
Sure go for it @subhamX :). Good luck! |
Hi @subhamX how has this been going? Did you get the help you need? |
The implementation part is almost done. I've tested the free APIs, but testing the PRO features is tricky (as I don't have Zoom PRO, and it's very expensive 😅). I've reached out to a few folks at Zoom, and their DevRel team might reply back soon. (in case they're interested in providing Zoom PRO for a day). Also, I had a small query on the design approach (and error_handling), and we're currently discussing here on airbytehq slack. Hopefully, in a few days, I shall make the PR. Please let me know if you have any suggestions or anything in general 🙂. |
Congrats on having this merged! |
@subhamX Can you please make sure your email is visible on your profile so we can contact you once we start sending payments out? Thank you! |
@RealChrisSean Done; |
No description provided.
The text was updated successfully, but these errors were encountered: