Skip to content
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

Update updatedAt timestamp after a integration check #1525

Merged
merged 3 commits into from
Sep 20, 2023
Merged

Conversation

yeganathan18
Copy link
Member

@yeganathan18 yeganathan18 commented Sep 20, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at 1341a75

Updated markIntegration function to track the last updated time of integrations. This is for enhancing the integration dashboard with more relevant information.

🤖 Generated by Copilot at 1341a75

markIntegration
Updates status and time stamp
A dashboard feature

Why

How

🤖 Generated by Copilot at 1341a75

  • Set updatedAt column of integrations table to current timestamp when updating integration status (link). This change improves the integration dashboard by showing the last updated time of each integration in services/apps/integration_run_worker/src/repo/integrationRun.repo.ts.

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.

@yeganathan18 yeganathan18 added the Bug Created by Linear-GitHub Sync label Sep 20, 2023
@yeganathan18 yeganathan18 merged commit 242aa32 into main Sep 20, 2023
8 checks passed
@yeganathan18 yeganathan18 deleted the bugfix/C-2316 branch September 20, 2023 20:20
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.

None yet

2 participants