-
-
Notifications
You must be signed in to change notification settings - Fork 264
Added the ability to edit the estimated hours of an issue #801
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
base: main
Are you sure you want to change the base?
Added the ability to edit the estimated hours of an issue #801
Conversation
There was a problem hiding this 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.
@@ -94,7 +94,7 @@ func GetMetadata() []*survey.Question { | |||
Name: "metadata", | |||
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"}, |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
Added the ability to edit the estimated hours of an issue after it has been created.