Skip to content

Process integration script fixed to work with the new integration framework#928

Merged
themarolt merged 10 commits intomainfrom
bugfix/process-integration-changes
Jun 1, 2023
Merged

Process integration script fixed to work with the new integration framework#928
themarolt merged 10 commits intomainfrom
bugfix/process-integration-changes

Conversation

@themarolt
Copy link
Copy Markdown
Contributor

@themarolt themarolt commented Jun 1, 2023

Changes proposed ✍️

What

  • delayed run processing in the new framework
  • fixed process-integration script so that it works with integrations in the new framework
  • fixed an issue with paginated processing in checkStuckIntegrationRuns.ts
  • changed how we trigger integration runs in the new framework to make it easier

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screehshots to the PR description for relevant FE changes
  • New backend functionality has been unit-tested.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.

@themarolt themarolt added the Bug Created by Linear-GitHub Sync label Jun 1, 2023
@themarolt themarolt marked this pull request as ready for review June 1, 2023 13:17
Copy link
Copy Markdown
Collaborator

@epipav epipav left a comment

Choose a reason for hiding this comment

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

lgtm

@themarolt themarolt merged commit 58a5410 into main Jun 1, 2023
@themarolt themarolt deleted the bugfix/process-integration-changes branch June 1, 2023 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Created by Linear-GitHub Sync

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants