Skip to content

Conversation

jcstorino
Copy link

Added the ability to edit the estimated hours of an issue after it has been created.

Copy link
Owner

@ankitpokhrel ankitpokhrel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @jcstorino Thank you for the PR and sorry for the late review. Most of the changes seems ok but we'd need to fix create flow before this could be merged.

Prompt: &survey.MultiSelect{
Message: "What would you like to add?",
Options: []string{"Priority", "Components", "Labels", "FixVersions", "AffectsVersions"},
Options: []string{"Priority", "Components", "Labels", "OriginalEstimate", "FixVersions", "AffectsVersions"},
Copy link
Owner

@ankitpokhrel ankitpokhrel Mar 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Edit seem to work fine but this will add OriginalEstimate for create command as well and we don't actually seem to process that. We'd need to fix original estimate in create flow.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @ankitpokhrel , now my time to say.. sorry for late repply.
The OriginalEstimate field is already part of the creation flow. It was included in the editing flow. Unless I am misunderstanding, the field already belongs to the creation flow.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I think I now understand the treatment for the field, I made the adjustments as informed.

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.

3 participants