Loading
Multiple issue assignees(BE)
What does this MR do?
Implements multiple assignees for issues
Why was this MR needed?
Screenshots (if relevant)
Backend TODO:
- Implement a migration to create table
- Implement a migration to migrate an old data
- Fix ability policies for multiple assignees
- Check all the migrations on staging
- Notifications
- System notes
- https://gitlab.com/gitlab-org/gitlab-ee/tree/master/app/services/issuable_base_service.rb#L6
- https://gitlab.com/gitlab-org/gitlab-ee/tree/master/app/views/notify/_reassigned_issuable_email.text.erb#L6
- Implement counts cache for issues (
update_assignee_cache_counts) - Create an issue about removing
issue.assignee_idfield until next release.(https://gitlab.com/gitlab-org/gitlab-ce/issues/30453) - Realations
- Unique index see Yorick's comment
- Split forms
- Chat commands
- Slash commands
- Fix for elasticsearch
- API v4
- API v3
- Final seacrh for
issue\.assignee[^s]to make sure every obvious case is taken care. - Issues export works
- Check out Importers
- Make sure DB seeds work
- Check issue board
- Check search (non-ES)
- Check filters
- Specs before applying new UI
- XML atom
- Make sure there is no
N+1problem - TODOs
- Make sure CE has appropriate specs that normally would fail on EE only. (From Sean:
however, I think it's really important that we can somehow test for this in CE, too. We really don't want to add a filter or a sort that only fails specs on EE) - We should warn users about changed API and Web Hooks(https://gitlab.com/gitlab-org/gitlab-ee/issues/2365)
- Create an issue: "We could allow unassigning individual people here, like the existing label commands
/label,/unlabel, and/relabel, but it doesn't need to happen now". (https://gitlab.com/gitlab-org/gitlab-ee/issues/2364)
Does this MR meet the acceptance criteria?
- Changelog entry added, if necessary
- Documentation created/updated
- API support added
- Tests
- Added for this feature/bug
- All builds are passing
- Conform by the merge request performance guides
- Conform by the style guides
- Branch has no merge conflicts with
master(if it does - rebase it please) - Squashed related commits together
What are the relevant issue numbers?
https://gitlab.com/gitlab-org/gitlab-ee/issues/1904
Performance testing
Data to seed database https://gitlab.com/gitlab-org/gitlab-ce/snippets/1657660