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

Milestone filter for pull requests shows either only open or closed milestones #11924

Closed
2 of 7 tasks
otbutz opened this issue Jun 17, 2020 · 8 comments · Fixed by #22423
Closed
2 of 7 tasks

Milestone filter for pull requests shows either only open or closed milestones #11924

otbutz opened this issue Jun 17, 2020 · 8 comments · Fixed by #22423
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented type/bug

Comments

@otbutz
Copy link

otbutz commented Jun 17, 2020

Description

While browsing open PRs i'm only able to select open milestones in the filter and vice-versa. This way i'm unable to search for old PRs which targeted a milestone which is closed now or search for open PRs targeting a closed milestone.

The milestone filter should behave similar to the dropdown which is used to select the target milestone of a pull request. It should display all milestones but could provide a visual separation for open/closed ones:

milestone-dropdown

Screenshots

open-prs
In this case i'm unable to filter open PRs targeting v1.0.0 because it's a closed milestone.

closed-prs
Here i'm unable to filter PRs targeting milestone v4.0.0 which is still open.

a1012112796 added a commit to a1012112796/gitea that referenced this issue Jun 20, 2020
@stale
Copy link

stale bot commented Aug 16, 2020

This issue has been automatically marked as stale because it has not had recent activity. I am here to help clear issues left open even if solved or waiting for more insight. This issue will be closed if no further activity occurs during the next 2 weeks. If the issue is still valid just add a comment to keep it alive. Thank you for your contributions.

@stale stale bot added the issue/stale label Aug 16, 2020
@6543
Copy link
Member

6543 commented Aug 16, 2020

ping - still here

@stale stale bot removed the issue/stale label Aug 16, 2020
@dexterxx-pl
Copy link

dexterxx-pl commented Feb 11, 2021

+1

v1.13.2 still affected.

@6543 6543 added type/enhancement An improvement of existing functionality issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented labels Feb 11, 2021
@otbutz
Copy link
Author

otbutz commented Feb 12, 2021

@6543 the label kind/enhancement doesn't really fit. This is a bug.

@lunny lunny added type/bug and removed type/enhancement An improvement of existing functionality labels Feb 18, 2021
@lunny
Copy link
Member

lunny commented Feb 18, 2021

So when filter on opened issues, we could just list opened milestones because closed milestones should not have opened issues. When filter on closed issues, we should list all milestones.

@otbutz
Copy link
Author

otbutz commented Feb 18, 2021

closed milestones should not have opened issues

As long as i can close a milestone with open issues, i should be able to filter accordingly. And closing a milestone could be also interpreted as not accepting new issues.

@lunny
Copy link
Member

lunny commented Feb 18, 2021

That sounds not a normal operation, once closed a milestone, the first step you should do is to move open issues to other milestone.

Not list all the milestones here because some repositories have many milestones and most of them closed.

@dexterxx-pl
Copy link

dexterxx-pl commented Feb 18, 2021

The case is when you add new milestone while you already have some closed issues and you want to assign them to it.

Eg.

You're starting development without milestones
Closing some issues while still working eg. on v1.0
Then you realize that you need milestones and you create such one for our WIP v1.0 version.

Already filtered milestones might be displayed below that (of current state) oppened/closed... just to be able to use them if I want.

6543 pushed a commit that referenced this issue Apr 30, 2023
Now we have `All milestones`, `No milestones`, `Open milestones` and
`Closed milestones`.
Fix #11924
Fix #22411 

<img width="1166" alt="image"
src="https://user-images.githubusercontent.com/81045/212243375-95eea035-a972-44b8-8088-53db614cb07e.png">
GiteaBot pushed a commit to GiteaBot/gitea that referenced this issue Apr 30, 2023
Now we have `All milestones`, `No milestones`, `Open milestones` and
`Closed milestones`.
Fix go-gitea#11924
Fix go-gitea#22411 

<img width="1166" alt="image"
src="https://user-images.githubusercontent.com/81045/212243375-95eea035-a972-44b8-8088-53db614cb07e.png">
silverwind pushed a commit that referenced this issue Apr 30, 2023
Backport #22423 by @lunny

Now we have `All milestones`, `No milestones`, `Open milestones` and
`Closed milestones`.
Fix #11924
Fix #22411 

<img width="1166" alt="image"
src="https://user-images.githubusercontent.com/81045/212243375-95eea035-a972-44b8-8088-53db614cb07e.png">

Co-authored-by: Lunny Xiao <[email protected]>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented type/bug
Projects
None yet
4 participants