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

auto-register blocks for google sheets data sources #270

Merged
merged 6 commits into from
Jan 6, 2025

Conversation

shekharnwagh
Copy link
Contributor

@shekharnwagh shekharnwagh commented Jan 3, 2025

Description

Note

JIRA: CAFE-1069

Auto-registers single and loop blocks for Google Sheets data sources added via the Remote Data Blocks Settings page. This brings it on par with Airtable integration. All fields are mapped to string types as Google Sheets sends tabular data in response without type information, nor does it strictly enforce types for cell values. A drawback of defaulting to string types is that fields with other types like image_url, numbers won't be handled properly and would be displayed as strings.

Testing

  • Start the local development environment (docs)
  • Add a Google Sheets data source via the Remote Data Blocks Settings page
  • Verify that the single and loop blocks are registered for the data source in the editor and they work as expected

@shekharnwagh shekharnwagh force-pushed the add/auto-register-google-sheets-blocks branch from 365f61f to 5549ccb Compare January 3, 2025 11:37
@shekharnwagh shekharnwagh self-assigned this Jan 3, 2025
@shekharnwagh shekharnwagh marked this pull request as ready for review January 3, 2025 15:54
Copy link
Contributor

@maxschmeling maxschmeling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works great! 🚢

@maxschmeling maxschmeling merged commit 1de044c into trunk Jan 6, 2025
11 checks passed
@maxschmeling maxschmeling deleted the add/auto-register-google-sheets-blocks branch January 6, 2025 08:23
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.

2 participants