-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[Enhancement]: Update Airtable Handler to be an API handler and use Python SDK #10091
Comments
@ZoranPandovski Hi, I created the issue for updating the Airtable Handler. |
Draft PR: #10108 |
@ZoranPandovski @MinuraPunchihewa Hi, for this issue, I am planning on not only modifying the handler to use the SDK but also transforming the Airtable handler from Database handler to an API (Application) handler because it seems more intuitive for me since it uses restful api calls. However, before proceeding further, I just want to discuss with you whether you agree on this because I think it is a big change and I don't want to delete/redo existing functionalities that may have some intended purposes. |
The Final decisionI will just make the handler to use the Python SDK ( |
Hey @HahaBill, |
No worries! Alright, I will then convert the handler into API handler. |
@ZoranPandovski @MinuraPunchihewa I've been struggling to make the SDK work in mindsdb, but finally, I have found a solution today: https://stackoverflow.com/questions/79234164/partially-initialized-module-pyairtable. It works! This requires the |
But anyway, I am almost done. I only need more testing! |
@HahaBill That's great! |
Alright, got it! |
@ZoranPandovski @MinuraPunchihewa Ready for a review! PR: #10108 I saw that there had been some updates regarding the Pydantic version, it seems to work fine now. |
Short description of current behavior
The current handler uses API endpoints to connect to Airtable. Using SDK is more preferable.
For more context: #9812 (comment)
Video or screenshots
No response
Expected behavior
No response
How to reproduce the error
No response
Anything else?
No response
The text was updated successfully, but these errors were encountered: