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

Fix various typings for FieldSet #390

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Conversation

sheindel
Copy link

@sheindel sheindel commented Oct 20, 2023

This actually nicely solves the Lookup problem by defining a FieldValue type and allowing it to be an array of FieldValue's

Fixes #389

All based on testing and AirTable field model docs

@sheindel sheindel changed the title Fix FieldValue for lookup field Fix various typings for FieldSet Oct 20, 2023
@sheindel
Copy link
Author

The only type I am not sure about is the AiText as I cannot test this. I have added the following types based on the docs, but I wanted to confirm with someone who works at AirTable first or whom has experience with this feature.

@sheindel
Copy link
Author

sheindel commented Mar 7, 2024

Just thought I'd bump this in case anyone at AirTable would like to merge this

@NabeelFarooqui-at
@BlakeThomson-at
@FredZhao-at

This should be incredibly quick to review and merge

@sheindel
Copy link
Author

sheindel commented Aug 1, 2024

Bumping this. It's a pretty simple change, could this be merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Potentially missing FieldSet type
1 participant