Skip to content

Store task errors following migrations and display them (#13246)#13287

Merged
lunny merged 1 commit intogo-gitea:release/v1.13from
techknowlogick:backport-13246
Oct 24, 2020
Merged

Store task errors following migrations and display them (#13246)#13287
lunny merged 1 commit intogo-gitea:release/v1.13from
techknowlogick:backport-13246

Conversation

@techknowlogick
Copy link
Copy Markdown
Member

  • Store task errors following migrations and display them

When migrate tasks fail store the error in the task table
and ensure that they show on the status page.

Fix #13242

Credit @zeripath

* Store task errors following migrations and display them

When migrate tasks fail store the error in the task table
and ensure that they show on the status page.

Fix go-gitea#13242

Signed-off-by: Andrew Thornton <art27@cantab.net>

* Update web_src/js/index.js

* Hide the failed first

Signed-off-by: Andrew Thornton <art27@cantab.net>

Co-authored-by: techknowlogick <techknowlogick@gitea.io>
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Oct 24, 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 Oct 24, 2020
@lunny lunny merged commit e177728 into go-gitea:release/v1.13 Oct 24, 2020
@techknowlogick techknowlogick deleted the backport-13246 branch October 24, 2020 05:03
@go-gitea go-gitea locked and limited conversation to collaborators Dec 14, 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.

4 participants