Skip to content

Conversation

jolheiser
Copy link
Member

Resolves #9748

In the past (before multi-select) I'm guessing this template check was because the selected repo would be the repo name for all the listed milestones.

If I missed something, please let me know. 😃

Signed-off-by: jolheiser <[email protected]>
@sapk sapk added type/bug topic/ui Change the appearance of the Gitea UI labels Jan 14, 2020
@sapk sapk added this to the 1.12.0 milestone Jan 14, 2020
Copy link
Member

@6543 6543 left a comment

Choose a reason for hiding this comment

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

works :)

@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Jan 14, 2020
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Jan 14, 2020
@6543
Copy link
Member

6543 commented Jan 14, 2020

@jolheiser created backport

@sapk
Copy link
Member

sapk commented Jan 14, 2020

It seems to always has been multi-select but it feel wrong to hide it.
Or we could only hide it when RepoIDs length = 1 ? no that good to always show it.

@sapk sapk added the backport/done All backports for this PR have been created label Jan 14, 2020
@6543
Copy link
Member

6543 commented Jan 14, 2020

I simply NOT hide it - to be consistent

@jolheiser
Copy link
Member Author

@jolheiser created backport

Thanks?

@codecov-io
Copy link

Codecov Report

Merging #9770 into master will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #9770      +/-   ##
==========================================
+ Coverage   42.32%   42.32%   +<.01%     
==========================================
  Files         600      600              
  Lines       78357    78357              
==========================================
+ Hits        33165    33168       +3     
+ Misses      41136    41134       -2     
+ Partials     4056     4055       -1
Impacted Files Coverage Δ
modules/process/manager.go 78.31% <0%> (+3.61%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d89022c...daf5693. Read the comment docs.

@sapk sapk merged commit cd5a44d into go-gitea:master Jan 14, 2020
@jolheiser jolheiser deleted the fix_milestone_list branch January 14, 2020 22:00
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport/done All backports for this PR have been created lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. topic/ui Change the appearance of the Gitea UI type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] on /milestones no repo-name
5 participants