Skip to content

Conversation

@cez81
Copy link
Contributor

@cez81 cez81 commented May 25, 2017

If xxx_active_code_live was < 60 minutes, it was printed as 0 hours. This PR will instead print it as days, hours and minutes.

Fixes #1716

@lunny lunny added the type/bug label May 26, 2017
@lunny lunny added this to the 1.2.0 milestone May 26, 2017
@lafriks
Copy link
Member

lafriks commented May 26, 2017

hours and minutes will be missing translation in other languages!

@tboerger tboerger added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label May 26, 2017
@lunny
Copy link
Member

lunny commented May 27, 2017

LGTM

@tboerger tboerger added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels May 27, 2017
@appleboy
Copy link
Member

LGTM

@tboerger tboerger 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 May 29, 2017
@appleboy appleboy merged commit b93568c into go-gitea:master May 29, 2017
@cez81 cez81 deleted the active_code_lives branch June 9, 2017 08:59
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Active codes lives < 60 minutes

5 participants