-
Notifications
You must be signed in to change notification settings - Fork 0
Feature/attio #69
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
Feature/attio #69
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds full-featured integration support for Attio by introducing helper functions, constants, types maps, and multiple actions for creating, fetching, updating, and finding records across various Attio endpoints. Key changes include:
- New helper functions and constants in the Attio subfolder.
- A series of localized actions for handling different API endpoints (object records, list entries, tasks, notes, etc.).
- Updates to the actions catalogue and CI workflow to incorporate the new Attio integration.
Reviewed Changes
Copilot reviewed 50 out of 51 changed files in this pull request and generated 1 comment.
Show a summary per file
File | Description |
---|---|
ts/src/apps/attio/helpers/constants.ts | Adds core helper functions and constants for the Attio API. |
ts/src/apps/attio/helpers/attio-types-map.ts | Introduces type mappings between Attio and Qorus types. |
ts/src/apps/attio/constants.ts | Provides constants such as app logo, API URL, and error class. |
ts/src/apps/attio/actions/*.ts | Implements several actions for create/update/get/find operations. |
ts/src/ActionsCatalogue/index.ts | Registers the Attio integration in the apps catalogue. |
.github/workflows/pull_request.yml | Updates CI workflow environment with an Attio token secret. |
Files not reviewed (1)
- ts/package.json: Language not supported
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR introduces a new Attio integration across the codebase by adding constants, type maps, various helper functions, and a suite of actions (create, update, get, and find) to interact with the Attio API.
- Added helper modules (constants, type maps) for Attio configuration.
- Implemented multiple actions for objects, lists, tasks, and notes.
- Updated the actions catalogue and CI workflow to incorporate the new Attio integration.
Reviewed Changes
Copilot reviewed 50 out of 51 changed files in this pull request and generated 1 comment.
Show a summary per file
File | Description |
---|---|
ts/src/apps/attio/helpers/constants.ts | Defines constants, helper functions, and error types for Attio API integration. |
ts/src/apps/attio/helpers/attio-types-map.ts | Maps Attio types to Qore types. |
ts/src/apps/attio/actions/*.ts | Implements actions for creating, updating, retrieving, and finding entities in Attio. |
ts/src/ActionsCatalogue/index.ts | Registers the new attio app into the catalogue. |
.github/workflows/pull_request.yml | Updates workflow environment variables to include ATTIO_TOKEN. |
Files not reviewed (1)
- ts/package.json: Language not supported
Comments suppressed due to low confidence (2)
ts/src/apps/attio/helpers/constants.ts:13
- [nitpick] The function name 'getAttioTokenRequired' might be clearer if renamed to 'requireAttioToken' to better indicate that it enforces the presence of a token.
export const getAttioTokenRequired = (
ts/src/apps/attio/actions/get-task.action.ts:49
- [nitpick] Consider preserving additional error details or logging the caught error before wrapping it in an AttioError, to aid in debugging.
throw new AttioError(`Failed to get task: ${error}`);
No description provided.