• Douwe Maan's avatar
    Merge branch 'todos-filter-project-delete' into 'master' · a834be61
    Douwe Maan authored
    Ensure we don't show TODOS for projects pending delete
    
    Joins the todos on the projects table in order to run the default scope. Also includes a where clause because the default scope is being removed soon.
    
    An alternative approach, more like the Issues page, would be to filter down the list by passing user.authorized_projects into the where clause.
    
    Or we could just be more defensive in the view when iterating.
    
    Todos page throws 500 error for users with todos in a project pending deletion.
    
    Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/17813
    
    cc\ @stanhu
    
    See merge request !4300
    a834be61
To find the state of this project's repository at the time of any of these versions, check out the tags.
CHANGELOG 144 KB