Repository navigation
Viewing issues and PRs in CLI should show the state (open, closed, merged) #652
Description
Describe the feature or problem you’d like to solve
Currently, issue view --preview shows who created an issue regardless of its state.
However, I think that what we actually want to see is the last event state such as Opened, Closed and Reopened (I guess Locked, Unlocked and Transferred could be included) and who made the last event.
Proposed solution
IssueTimelineItemsItemType provides various issue events and we can extract the last event.
I gave a shot with 58f6be...doi-t:cf6e746 and it seems it is possible to do like below. These captures show the last state and its actor instead of the author of the issue.
If it makes sense to everyone and is fine with contributing guide, I'd happy to work on it with unit tests for each event.
Activity
billygriffin commentedon Mar 13, 2020
Thanks @doi-t! I'd love to hear @ampinsk's thoughts, but in trying out several things, I think the biggest missing piece of context is the current state, not the last event. For example, when I'm viewing an issue on the web, its status is unambiguous.
However, that same issue, when viewing it in terminal, looks like this.
And even when viewing the whole preview in terminal, there's no way to see that the issue I'm looking at is actually closed. That strikes me as one of the most critical pieces of context when viewing an issue. So in my view, it's less about what the last piece of activity was (because that's ambiguous in terms of what information it's conveying), and more about the issue or PR's current status.
I don't have a strong opinion about how to convey that information and would leave that to @ampinsk to make a call on, but I do think it should be prominent. Curious your thoughts.
ampinsk commentedon Mar 13, 2020
Agree with @billygriffin's points here! Basically, "opened by" really means the author and that should stay consistent. But if we want the state to be clearer, and to be more parallel with github.com, I think we could do that like so:
We can also do this for PRs, but keep the metadata on PRs the same as it is now
Note: This change means removing labels from this header
[-]`issue view --preview` should show the last event[/-][+]Viewing issues and PRs in CLI should show the state (open, closed, merged)[/+]doi-t commentedon Mar 14, 2020
I see! It is simpler than what I thought and unambiguous as I wanted. 👍 👍 I understand that
view --previewis based on the title and description with some metadata at the top on the web.If it's ready to go and fine to take it, I can work on it. 🙋♂️
billygriffin commentedon Mar 16, 2020
@doi-t We discussed this this morning, and if you'd like to do this, please go ahead. One note is that "Merged" (in purple) should be a possible state for PRs as well. Thanks!
Remove labels from the issue preview header
doi-t commentedon Mar 18, 2020
@ampinsk
I've added the Draft PR state in #667 which was not mentioned in #652 (comment). I picked Gray color as same as
gh pr status. Please check it out along with other designs.In addition, is it expected to use
•(Bullet:U+2022) as a separator?ampinsk commentedon Mar 18, 2020
Perfect, sounds good!
Yes, exactly. Thank you!