Skip to content

Conversation

themarolt
Copy link
Contributor

@themarolt themarolt commented Oct 12, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at 6c63496

The pull request improves the integration tick processor by adding an order clause to the query that fetchs all integrations from integrationRepository.ts and a logging statement to the processor logic in integrationTickProcessor.ts. These changes enhance the consistency and visibility of the integration execution.

🤖 Generated by Copilot at 6c63496

Tick processor
Sorts and skips integrations
Logs with clarity

Why

How

🤖 Generated by Copilot at 6c63496

  • Order integrations by id to ensure consistent processing (link)
  • Log when an integration is skipped due to an existing run (link)

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.

@garrrikkotua garrrikkotua self-requested a review October 12, 2023 13:01
@themarolt themarolt merged commit 4b8e0d7 into main Oct 12, 2023
@themarolt themarolt deleted the remove-nodejs-worker-check-integrations-step branch October 12, 2023 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants