kaije / toggl-to-jira Goto Github PK

The worklog id is only a partial key for the worklog record in Jira - should really capture the returned issueIdOrKey as well.

Refer to docs at:
https://developer.atlassian.com/cloud/jira/platform/rest/v2/api-group-issue-worklogs/#api-rest-api-2-issue-issueidorkey-worklog-id-get

At the moment, the script can only be run once per target date, or the user risks sending duplicate work log entries to Jira.

As each Toggl entry has a unique id, we could maintain a log of entries already sent to Jira somewhere on the user's machine (e.g. a local file? app data?).

The Temporal ECMAScript proposal is still only at stage 2. Consider using one of the recommended alternatives in the meantime?

https://momentjs.com/docs/#/-project-status/recommendations/
https://tc39.es/proposal-temporal/docs/index.html

Feature: Print a duration in seconds in the format "0h 0m 0s" or "0h 0m".

Currently used when listing found Toggl entries for review by the user.

Should also use this format in the message displayed when an entry is sent to Jira, rather than displaying the duration in seconds:

Logging 312 seconds to Jira issue SDDSD-3917: Work on Toggl to Jira script...
https://qut.atlassian.net/rest/api/2/issue/SDDSD-3917/worklog
Success!

Will also want to use this formatting logic when the 'round to nearest minute' feature is implemented.

e.g.

(XYZ-3917) Work on Toggl to Jira script - 0h 5m 12s (Will be rounded to 0h 5m)
(XYZ-3917) Work on Toggl to Jira script - 0h 0m 4s  (Will be rounded to 0h 1m)
(ABC-123) Wrote some docs - 1h 07m 39s (Will be rounded to 1h 8m)

At least output the Jira worklog id?

Consider logging to a file?

Consider appending sent entry to file rather than rewriting entire file contents each time.

Should this be optional?
Prompt the user?

There are a couple of relevant aspects:

a) Jira will not accept any work log of less than 1 minute (returns a 400 Bad Request response)

b) Jira only logs hours and minutes - leftover seconds are discarded. This means that if you have a Toggl time entry of 4m 59s, it will be logged in Jira as 4 minutes. (The missing seconds will add up over time, so it seems worth rounding up where the remaining seconds are greater than 30.)

An error occurs when running npm run lint following the recent updates to dependencies:

ESLint: 7.32.0

Error: Cannot read config file: C:\Users\Kelly\dev\repos\toggl-to-jira\node_modules\[email protected]
Error: "prettier/@typescript-eslint" has been merged into "prettier" in eslint-config-prettier 8.0.0. See: https://github.com/prettier/eslint-config-prettier/blob/main/CHANGELOG.md#version-800-2021-02-21
Referenced from: C:\Users\Kelly\dev\repos\toggl-to-jira.eslintrc
at Object. (C:\Users\Kelly\dev\repos\toggl-to-jira\node_modules\[email protected]:1:69)
...