Skip to content

Conversation

@harrytwigg
Copy link
Collaborator

@harrytwigg harrytwigg commented Apr 28, 2025

Disables the updating of individual time entries from timesheeter inside of toggl.

We shouldn't need this functionality as toggl should be the source of truth for time entries

Testing instructions:

  1. Create a toggl integration on staging to a test workspace
  2. Create a time entry
  3. Wait for time entry to get synced to timesheeter
  4. Update the time entry start end times from the timesheeter gui
  5. The time entry will not get updated inside toggl

@harrytwigg harrytwigg requested review from Copilot and tomaske April 28, 2025 10:39
Copy link

Copilot AI left a 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 pull request disables the time entry update functionality in Toggl from Timesheeter to ensure that Toggl remains the source of truth for time entries. Key changes include:

  • Removing the axios PUT call for updating time entries.
  • Adding a console log that indicates update attempts are disabled.
  • Commenting out the original response validation code.
Comments suppressed due to low confidence (1)

packages/backhouse/src/integrations/toggl/api/time-entries.ts:70

  • The log message uses 'toggle' instead of 'Toggl', which could be confusing when correlating log entries with the integration. Consider updating 'toggle' to 'Toggl'.
console.log(`Attempted to update time entry in toggle but this is disabled for now: ${path.time_entry_id}: ${JSON.stringify(body)}`);

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