Skip to content

Conversation

lafriks
Copy link
Member

@lafriks lafriks commented Mar 12, 2018

Backport #3661

@lafriks lafriks added this to the 1.4.0 milestone Mar 12, 2018
@lunny
Copy link
Member

lunny commented Mar 13, 2018

LGTM

@tboerger tboerger added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Mar 13, 2018
@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 Mar 13, 2018
@lafriks lafriks merged commit 0c61212 into go-gitea:release/v1.4 Mar 13, 2018
@lafriks lafriks deleted the backport/oauth2_goth branch March 13, 2018 07:08
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
project-mirrors-bot-tu bot pushed a commit to project-mirrors/forgejo-as-gitea-fork that referenced this pull request Jan 23, 2025
When rendering templates for packages, be more forgiving about missing metadata. For some repository types - like maven - metadata is uploaded separately. If that upload fails, or does not happen, there will be no metadata. In that case, Forgejo should handle it gracefully, and render as much of the information as possible, without erroring out. Rendering without metadata allows one to delete a partial package, while if we throw errors, that becomes a whole lot harder. This patch adjusts the generic metadata template, and also the maven template. There may be more cases of the same problem lying around. Fixes go-gitea#3663. Signed-off-by: Gergely Nagy <forgejo@gergo.csillger.hu> (cherry picked from commit ac4d535)
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

4 participants