Skip to content

Conversation

jolheiser
Copy link
Member

Fixes #6721

Previously assignees on the issue were nil since they weren't loaded, which messed up any ranging that happened when comparing.

Signed-off-by: jolheiser <john.olheiser@gmail.com>
Signed-off-by: jolheiser <john.olheiser@gmail.com>
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Apr 23, 2019
@adelowo
Copy link
Member

adelowo commented Apr 23, 2019

Nice catch... We really need more tests for the api handlers

@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 Apr 23, 2019
@techknowlogick
Copy link
Member

@jolheiser thanks for PR :) I'll be merging this once build is complete, could you create a backport PR once I merge this one?

@jolheiser
Copy link
Member Author

@techknowlogick Sure thing! 😄

@techknowlogick techknowlogick added the modifies/api This PR adds API routes or modifies them label Apr 23, 2019
@codecov-io
Copy link

codecov-io commented Apr 23, 2019

Codecov Report

Merging #6723 into master will increase coverage by 0.01%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6723      +/-   ##
==========================================
+ Coverage   40.72%   40.73%   +0.01%     
==========================================
  Files         421      421              
  Lines       57867    57872       +5     
==========================================
+ Hits        23567    23575       +8     
+ Misses      31171    31167       -4     
- Partials     3129     3130       +1
Impacted Files Coverage Δ
routers/api/v1/repo/issue.go 23.22% <0%> (-0.25%) ⬇️
routers/repo/view.go 42.6% <0%> (+1%) ⬆️
models/repo_list.go 67.89% <0%> (+1.05%) ⬆️
models/unit.go 51.42% <0%> (+5.71%) ⬆️

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 cbf8538...c0485d4. Read the comment docs.

@techknowlogick techknowlogick merged commit 45fa5cc into go-gitea:master Apr 23, 2019
@lafriks
Copy link
Member

lafriks commented Apr 23, 2019

Please create backport to 1.8 branch

jolheiser added a commit to jolheiser/gitea that referenced this pull request Apr 23, 2019
@jolheiser jolheiser deleted the 6721_delete_assignee_api branch April 23, 2019 17:19
@techknowlogick techknowlogick added the backport/done All backports for this PR have been created label Apr 23, 2019
@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. modifies/api This PR adds API routes or modifies them type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[API] Issue edit API does not replace assignee
6 participants