Skip to content

Commit

Permalink
message
Browse files Browse the repository at this point in the history
  • Loading branch information
andyh565 committed Oct 15, 2024
1 parent 91c7ecc commit 9f30239
Showing 1 changed file with 0 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -121,14 +121,6 @@ def _populate_fields_meta(self, module: ModuleMeta):
fields_meta_json = self.api.fields_settings(module.api_name)
fields_meta = []
for field in fields_meta_json:
'''
Skipping this field for now, as it is a bug in Zoho's V2 API.
The alternative is to update the API version to v5, but then the number of fields that can be fetched per record is limited to 50.
If the API version is updated to v5, we will need to do one of the following:
- make multiple API requests to fetch the complete record data (req 1: get first 50 fields, req 2: get next 50 fields), then aggregate the records.
- fetch field data using v5, fetch records using v2 (mixing API versions is a bad idea)
- pass in the fields as connection params that the user wants to fetch and limit it to 50 fields
'''
if 'json_type' not in field:
logger.warn(f"Unsupported Zoho field {field['api_name']} found")
continue
Expand Down

0 comments on commit 9f30239

Please sign in to comment.