Skip to content

feature(cSDK): ConfigurationForm and test example#576

Open
fivetran-anushkaparashar wants to merge 27 commits into
mainfrom
RD-1063108_config_form_test_example
Open

feature(cSDK): ConfigurationForm and test example#576
fivetran-anushkaparashar wants to merge 27 commits into
mainfrom
RD-1063108_config_form_test_example

Conversation

@fivetran-anushkaparashar

@fivetran-anushkaparashar fivetran-anushkaparashar commented Apr 23, 2026

Copy link
Copy Markdown
Contributor

Jira ticket

Closes https://fivetran.atlassian.net/browse/RD-1063108

Description of Change

Added an example for connector sdk that defines configuration form and tests

Testing

Screenshot 2026-04-23 at 10 38 02 PM

Checklist

Some tips and links to help validate your PR:

  • Tested the connector with fivetran debug command.
  • Added/Updated example-specific README.md file, see the README template for the required structure and guidelines.
  • Followed Python Coding Standards, refer here

@github-actions

github-actions Bot commented Apr 23, 2026

Copy link
Copy Markdown

🧹 Python Code Quality Check

✅ No issues found in Python Files.

🔍 See how this check works

This comment is auto-updated with every commit.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a new quickstart example connector demonstrating how to define a ConfigurationForm (with multiple field types) and register a “Test Connection” setup test for Connector SDK users.

Changes:

  • Introduces configuration_form() with fields and a connection_test() setup test.
  • Adds a paginated sync example in update() that fetches /posts and checkpoints an offset cursor.
  • Adds an example configuration.json for local debugging.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 9 comments.

File Description
examples/quickstart_examples/configuration_form/connector.py New example connector showcasing configuration form + test connection + paginated sync loop
examples/quickstart_examples/configuration_form/configuration.json Adds a sample config for the new example

Comment thread examples/quickstart_examples/configuration_form/connector.py Outdated
Comment thread examples/quickstart_examples/configuration_form/connector.py Outdated
Comment thread examples/quickstart_examples/configuration_form/connector.py Outdated
Comment thread examples/quickstart_examples/configuration_form/configuration.json Outdated
Comment thread examples/quickstart_examples/configuration_form/connector.py Outdated
Comment thread examples/quickstart_examples/configuration_form/connector.py Outdated
Comment thread examples/quickstart_examples/configuration_form/connector.py Outdated
Comment thread examples/quickstart_examples/configuration_form/connector.py
Comment thread examples/quickstart_examples/configuration_form/connector.py
@fivetran-anushkaparashar fivetran-anushkaparashar marked this pull request as ready for review April 23, 2026 17:40
@fivetran-rishabhghosh fivetran-rishabhghosh requested review from fivetran-sahilkhirwal and fivetran-satvikpatil and removed request for a team April 26, 2026 17:28
Comment thread examples/quickstart_examples/configuration_form/README.md Outdated
Comment thread examples/quickstart_examples/configuration_form/README.md Outdated
Co-authored-by: Dejan Tucakov <dejan.tucakov@fivetran.com>

@fivetran-dejantucakov fivetran-dejantucakov left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@fivetran-anushkaparashar Approved with. minor formatting updates

@fivetran-JenasVimal fivetran-JenasVimal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGMT

@fivetran-dejantucakov fivetran-dejantucakov left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@fivetran-anushkaparashar

Copy link
Copy Markdown
Contributor Author

@fivetran-sahilkhirwal @fivetran-dejantucakov Can you please re-approve? Your review was dismissed by a latest small change that I pushed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants