Skip to content

[CON-3149] feat(clio): write functionality#2956

Open
Lafetz wants to merge 1 commit into
clio-metadatafrom
clio-write
Open

[CON-3149] feat(clio): write functionality#2956
Lafetz wants to merge 1 commit into
clio-metadatafrom
clio-write

Conversation

@Lafetz
Copy link
Copy Markdown
Contributor

@Lafetz Lafetz commented May 6, 2026

Conventions

Read more: https://ampersand.slab.com/posts/deep-connectors-guide-6x4fhxne#ht0ds-reviewer-checklist

  • Connector uses internal/components
  • Metadata uses V2 metadata format
  • Read supports pagination and incremental sync
  • Raw response is returned as is for READ & METADATA, no formatting or flattening is performed.
  • Write payloads should accept what ReadResults.Fields is returning. Any unnecessary nesting around the input is removed.
  • Provider errors are mapped if non-standard (errors with 200 response code are converted to 4XX)
  • Custom fields, if not human readable names, are resolved to readable names.
  • Unit tests cover read/write/metadata logic (placed in /tests/)
  • Appropriate object names are used. Objects need to be resources, not actions (jobs and not jobs.list).

Tests

image image

@linear
Copy link
Copy Markdown

linear Bot commented May 6, 2026

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.

1 participant