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

add possibility to edit a worklog #45

Open
sebmeyer opened this issue Dec 15, 2020 · 5 comments
Open

add possibility to edit a worklog #45

sebmeyer opened this issue Dec 15, 2020 · 5 comments
Labels
enhancement New feature or request

Comments

@sebmeyer
Copy link

Currently, whenever there is an issue with a worklog, I need to delete and recreate it.
It would be great if a worklog could be edited (description, start time, duration, ...) to make it easier to fix typos or mistakes in initial logs.

@szymonkozak szymonkozak added the enhancement New feature or request label Dec 15, 2020
@szymonkozak
Copy link
Owner

Do you have any propositions about the usage? Interactive command or something else? I don't have any idea for good design for this feature. It would be easy to add something like tempo edit 123 --description "new description", but I don't know if you mean that.

@sebmeyer
Copy link
Author

Yeah, that's what i envisioned to be honest
tempo edit <worklog_id> with options for the attributes that are reasonably likely to be edited:

  • issue key or alias
  • duration or start/end
  • description

In my opinion an interactive command would be messier than this, because it'd have to constantly ask what the user wants to edit.
If I can come up with a better solution I'll comment again.

@Vivec1990
Copy link

I have tried implementing this.
Is it not possible to move a worklog to another issue via the rest API? If i send an update request as defined in the tempo api with another issue key than the one already associated with the worklog, it always tells me that the issue does not exist (but it does).

@szymonkozak do you have any experience with this?

@szymonkozak
Copy link
Owner

@Vivec1990 I have the same experience as you. It looks like a bug in the API, you can report it here: https://tempo-io.atlassian.net/servicedesk/customer/portal/6

They are pretty responsive.

I don't have too much time until February/March, so if you want you can implement this feature and open Pull Request.

@Vivec1990
Copy link

Okay, so the answer to the inquiry was that this is sadly expected behavior.
As the edit worklog request would take basically the same arguments as adding a new worklog, i'll just go the way that tempo ui is apparently going and delete the old worklog and add a new one.

I'll open a pr for this once it is done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants