feat: re-add Linear integration #31114
Conversation
…g into err-dn/linear-integration
…g into err-dn/linear-integration
…g into err-dn/linear-integration
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
PR Summary
This PR reintroduces Linear integration support for error tracking in PostHog, reverting a previous revert while maintaining proper migration numbering.
- Added
LinearTeamPickercomponent andlinearIntegrationLogicfor selecting and managing Linear teams - Added new
ErrorTrackingIntegrationscomponent to handle Linear workspace connections - Implemented Linear template for creating issues from PostHog events with proper GraphQL integration
- Added migration
0709_alter_integration_kind.pyto support 'linear' as a valid integration kind - Added environment variables
LINEAR_APP_CLIENT_IDandLINEAR_APP_CLIENT_SECRETfor OAuth configuration
19 file(s) reviewed, 2 comment(s)
Edit PR Review Bot Settings | Greptile
| linearIntegrations: [ | ||
| (s) => [s.integrations], | ||
| (integrations) => { | ||
| return integrations?.filter((x) => x.kind == 'linear') || [] |
There was a problem hiding this comment.
style: The fallback to empty array is only used for linearIntegrations but not for slackIntegrations (line 177). Consider adding the same fallback pattern to slackIntegrations for consistency.
| return integrations?.filter((x) => x.kind == 'linear') || [] | |
| return integrations?.filter((x) => x.kind == 'slack') || [] |
| if (res.status != 200 or res.body.success == false) { | ||
| throw Error(f'Failed to create Linear issue: {res.status}: {res.body}'); |
There was a problem hiding this comment.
logic: The error check res.body.success == false assumes the response body has a success property, but the GraphQL response structure might not include this directly. Should check for res.body.data?.issueCreate?.success instead.
|
Size Change: +243 B (0%) Total Size: 13.2 MB ℹ️ View Unchanged
|
Problem
Revert of the revert in #31112
Changes
Nothing changes from #30917 except the migration number is one lower because the bad migration has been removed