example(connector_sdk) : re-raised new connector by fivetran-JenasVimal · Pull Request #39 · fivetran/community_connectors

fivetran-JenasVimal · 2026-07-02T10:32:47Z

Description of Change

Re-raising this connector from old repo

RD-1237871

Testing

This is actively being used by a customer. Also, tested locally this morning here

Checklist

Some tips and links to help validate your PR:

Tested the connector with fivetran debug command.
Added/Updated example specific README.md file, refer here for template.
Followed Python Coding Standards, refer here

Copilot

Pull request overview

Adds a new people_ai Connector SDK example (ported from an older repo) that authenticates to the People.ai API via OAuth2 client credentials, paginates through activities endpoints, and upserts results into destination tables.

Changes:

Added people_ai/connector.py implementing OAuth token acquisition, retry/backoff, pagination, and upserts.
Added people_ai/README.md documenting usage, configuration, and produced tables.
Added people_ai/configuration.json template plus a (currently empty) people_ai/requirements.txt.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 22 comments.

File	Description
people_ai/connector.py	Implements the People.ai API sync logic (auth, pagination, retries, upserts, checkpointing).
people_ai/README.md	Documents connector behavior, setup, configuration parameters, and tables.
people_ai/configuration.json	Provides placeholder configuration keys for API credentials.
people_ai/requirements.txt	Added file intended for dependencies (currently empty).

+        except requests.exceptions.HTTPError as e:
+            # The retry logic is now inside get_page,
+            # so an error here means it failed permanently
+            log.severe(f"Permanent failure fetching base activities" f" at offset {offset}: {e}")


fivetran-sahilkhirwal

changes look good
For tests, please rebase after this is merged: #30

Copilot

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 24 comments.

Copilot

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 23 comments.

Copilot

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 22 comments.

This reverts commit b96ce1b.

Copilot

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 12 comments.

Copilot

Pull request overview

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

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 7 comments.

Copilot

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 8 comments.

fivetran-JenasVimal · 2026-07-07T11:13:09Z

Was unable to test this as I dont have credentials .

Verified the People.ai endpoints are valid/routable. The activity endpoints return 401 Authentication exception without credentials, which confirms the routes exist and require auth rather than returning 404. The token endpoint returns 405 Method Not Allowed for GET, but returns 401 Invalid credentials for the expected POST form request with dummy credentials, matching the connector’s OAuth client credentials flow.

Copilot

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 8 comments.

+    Returns:
+        int: The total number of records successfully processed (upserted).
+    """
+    offset = 0


+        if len(page) < limit:
+            break
+
+        offset += limit


+        state["activity_offset"] = offset + len(page)
+        op.checkpoint(state)
+
+        if len(page) < limit:
+            break
+
+        offset += limit


+            except Exception as e:
+                log.error(f"Failed to refresh token: {e}")
+                raise e  # If re-auth fails, raise the exception


+        # Save the progress by checkpointing the state. This is important for ensuring that the sync process can resume
+        # from the correct position in case of next sync or interruptions.
+        # Learn more about how and where to checkpoint by reading our best practices documentation
+        # (https://fivetran.com/docs/connectors/connector-sdk/best-practices#largedatasetrecommendation).
+        state["activity_offset"] = offset + len(page)
+        op.checkpoint(state)


+    required_configs = ["api_key", "api_secret"]
+    for key in required_configs:
+        if key not in configuration:
+            raise ValueError(f"Missing required configuration value: {key}")


+    try:
+        access_token = get_access_token(api_key, api_secret)
+    except requests.exceptions.RequestException as e:
+        log.error(f"FATAL: Initial authentication failed: {e}")
+        return  # Stop the sync


 - **[oura_ring](https://github.com/fivetran/community_connectors/tree/main/oura_ring)** - This example shows how to sync health and wellness data from the Oura Ring API v2 using Connector SDK. It syncs daily activity, sleep, readiness, stress, and heart rate data with incremental syncing, cursor-based pagination, automatic flattening of nested contributor objects, and date-range chunking for high-volume heart rate data. You need to provide your Oura Personal Access Token for this example to work.
 - **[owasp_api_vulns](https://github.com/fivetran/community_connectors/tree/main/owasp_api_vulns)** - Sync OWASP API vulnerability data from NVD 2.0
 - **[partech](https://github.com/fivetran/community_connectors/tree/main/partech)** - Sync POS data from Partech (formerly Punchh)
+- **[people_ai](https://github.com/fivetran/community_connectors/tree/main/people_ai)** - Sync activity and participant data from People.ai using OAuth2 client credentials


fivetran-satvikpatil · 2026-07-07T18:57:53Z

+    try:
+        access_token = get_access_token(api_key, api_secret)
+    except requests.exceptions.RequestException as e:
+        log.error(f"FATAL: Initial authentication failed: {e}")


Why are we silently skipping here? We need to fail if we are unable to authenticate. On line 188, we call this method and assign the return value to : current_token = reauth_func()
Instead should we directly fail?

fivetran-satvikpatil · 2026-07-07T18:59:54Z

+            the specified activity type.
+    """
+    state_offset_key = f"{activity_type}_offset"
+    offset = state.get(state_offset_key, 0)


The key read here is different than the key that is being checkpointed on line 357, how is it consistent?

fivetran-satvikpatil · 2026-07-07T19:00:37Z

@codex review

fivetran-satvikpatil

Please address all the comments given by the copilot/codex.

fivetran-satvikpatil · 2026-07-07T19:05:30Z

+- `schema()` defines the `participants` table with primary key `uid`, `email`.
+- `sync_base_activities()` copies each base activity record and renames a `subject` field to `api_subject` when present.
+- `sync_activity_type()` writes participant records to the `participants` table.
+- The connector stores the most recently processed page offsets in `state["activity_offset"]` and `state["participants_offset"]` after each successful page write.


Where are we using state["participants_offset"]?

Copilot AI review requested due to automatic review settings July 2, 2026 10:32

Copilot started reviewing on behalf of fivetran-JenasVimal July 2, 2026 10:33 View session

fivetran-JenasVimal mentioned this pull request Jul 2, 2026

example(PeopleAI): add people_ai connector example fivetran/connector_sdk#358

Closed

3 tasks

fivetran-JenasVimal self-assigned this Jul 2, 2026

Copilot AI reviewed Jul 2, 2026

View reviewed changes

fivetran-sahilkhirwal previously approved these changes Jul 2, 2026

View reviewed changes

fivetran-JenasVimal requested a review from a team as a code owner July 3, 2026 07:49

Copilot AI review requested due to automatic review settings July 3, 2026 07:53

fivetran-JenasVimal dismissed fivetran-sahilkhirwal’s stale review via 8c3fc1b July 3, 2026 07:53

fivetran-JenasVimal force-pushed the re-raise_people-ai branch from 0523275 to 8c3fc1b Compare July 3, 2026 07:53

Copilot started reviewing on behalf of fivetran-JenasVimal July 3, 2026 07:54 View session

Copilot AI reviewed Jul 3, 2026

View reviewed changes

Copilot AI review requested due to automatic review settings July 3, 2026 08:06

Copilot started reviewing on behalf of fivetran-JenasVimal July 3, 2026 08:06 View session

Copilot AI reviewed Jul 3, 2026

View reviewed changes

Copilot AI review requested due to automatic review settings July 3, 2026 08:12

Copilot started reviewing on behalf of fivetran-JenasVimal July 3, 2026 08:13 View session

fivetran-rishabhghosh removed the request for review from a team July 3, 2026 08:14

Copilot AI reviewed Jul 3, 2026

View reviewed changes

fivetran-JenasVimal added 5 commits July 3, 2026 15:09

re-raised new connector

5935c36

Update People.ai connector listing

3124427

Handle label permission failures

bc141cb

Revert "Handle label permission failures"

1227113

This reverts commit b96ce1b.

Format People.ai connector

fd7caa1

fivetran-JenasVimal force-pushed the re-raise_people-ai branch from 3e46920 to fd7caa1 Compare July 3, 2026 09:39

github-actions Bot added size/L ai-assisted/unknown risk/unknown labels Jul 3, 2026

fivetran-JenasVimal requested a review from fivetran-sahilkhirwal July 6, 2026 05:36

fivetran-sinchanahegde previously approved these changes Jul 6, 2026

View reviewed changes

Copilot started reviewing on behalf of fivetran-JenasVimal July 7, 2026 09:03 View session

Copilot AI reviewed Jul 7, 2026

View reviewed changes

fivetran-JenasVimal added 4 commits July 7, 2026 14:44

changed logs

64f754f

Made changes to code

7907eff

Address People.ai README review comments

f5c4937

Format People.ai connector

b405c50

Copilot AI review requested due to automatic review settings July 7, 2026 10:37

Copilot started reviewing on behalf of fivetran-JenasVimal July 7, 2026 10:37 View session

Copilot AI reviewed Jul 7, 2026

View reviewed changes

Potential fix for pull request finding

1d706bc

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings July 7, 2026 10:52

Copilot started reviewing on behalf of fivetran-JenasVimal July 7, 2026 10:52 View session

fivetran-JenasVimal and others added 4 commits July 7, 2026 16:22

Potential fix for pull request finding

49b7892

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Potential fix for pull request finding

daf0775

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Potential fix for pull request finding

5886608

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Potential fix for pull request finding

29934a3

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI reviewed Jul 7, 2026

View reviewed changes

Comment thread people_ai/connector.py

Comment thread people_ai/connector.py

Comment thread people_ai/connector.py

Comment thread people_ai/connector.py

Comment thread people_ai/connector.py

Comment thread people_ai/connector.py

Comment thread people_ai/connector.py Outdated

Copilot AI review requested due to automatic review settings July 7, 2026 10:56

Copilot started reviewing on behalf of fivetran-JenasVimal July 7, 2026 10:56 View session

Copilot AI reviewed Jul 7, 2026

View reviewed changes

Apply repository Python formatting

a489ddd

fivetran-sinchanahegde requested a review from Copilot July 7, 2026 11:18

Copilot started reviewing on behalf of fivetran-sinchanahegde July 7, 2026 11:18 View session

fivetran-sinchanahegde approved these changes Jul 7, 2026

View reviewed changes

Copilot AI reviewed Jul 7, 2026

View reviewed changes

fivetran-satvikpatil reviewed Jul 7, 2026

View reviewed changes

Uh oh!

Conversation

fivetran-JenasVimal commented Jul 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description of Change

Testing

Checklist

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

fivetran-sahilkhirwal left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

fivetran-JenasVimal commented Jul 2, 2026 •

edited

Loading