Search for issues with multiple assignees with implicit AND condition
Resources
PM @victorwu | UX @cperessini | FE @alfredo1
Docs blurb
Docs should be incorporated together with: https://gitlab.com/gitlab-org/gitlab-ee/issues/1904.
Description
With multiple assignees (https://gitlab.com/gitlab-org/gitlab-ce/issues/13386), this is how searching will behave:
I search for:
assignee: @victorwu assignee: @ClemMakesApps
I get all issues that are assigned to both @victorwu
+ @ClemMakesApps
. This includes issues that have other assignees. But all the results have at least these two assignees.
Designs
- Show closed items
Activity
-
Newest first Oldest first
-
Show all activity Show comments only Show history only
- Victor Wu changed milestone to gitlab-ce%21
changed milestone to gitlab-ce%21
- Victor Wu added gitlab-ce~992791 label
added gitlab-ce~992791 label
- Victor Wu added gitlab-ce
874211 gitlab-ce127616 labelsadded gitlab-ce
874211 gitlab-ce127616 labels @ClemMakesApps @cperessini : Is this what we want, per our discussions in https://gitlab.com/gitlab-org/gitlab-ce/issues/13386#note_24109961?
I search for:
assignee: @victorwu assignee: @ClemMakesApps
I get all issues that are assigned to both
@victorwu
+@ClemMakesApps
. This includes issues that have other assignees. But all the results have at least these two assignees.- Victor Wu changed milestone to gitlab-ce%35
changed milestone to gitlab-ce%35
- Maintainer
yes I think that's right. It should be a similar behavior as searching for
label:~UX label:~Frontend
in our current implementation imo Thanks @ClemMakesApps : updated desc.
- Developer
@victorwu @ClemMakesApps just confirming the described behavior is correct
Thanks @cperessini !
- username-removed-419655 added gitlab-ce~14281 label
added gitlab-ce~14281 label
- username-removed-408881 assigned to @alfredo1
assigned to @alfredo1
- Contributor
@victorwu how is this going to work on CE?. It looks like multiple assignees it's going to be a EE feature.
- Victor Wu moved from gitlab-ce#28947
moved from gitlab-ce#28947
Thanks @alfredo1 . Sorry my mistake in not updating this to EE earlier.
For the CE scenario, @ClemMakesApps, this will prevent the user from encountering that scenario right? https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/9406
cc @cperessini
- Maintainer
I think this will need to be added into CE codebase even though the feature is only available in EE because we are going to add multiple assignee code to both codebases but restrict the selection in CE
- username-removed-408881 mentioned in merge request !1494 (merged)
mentioned in merge request !1494 (merged)
- Contributor
I've created a WIP MR here https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/1494. Backend and some feature tests are left. Is there a progress in multiple assignee feature @vsizov?
- Contributor
@alfredo1 You can track the progress here https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/1480 I haven't spent too much time on this yet but I'm working on it right now.
Edited by Valery Sizov - username-removed-408677 mentioned in issue #1904 (closed)
mentioned in issue #1904 (closed)
- Contributor
The Multiple Assignee Feature is already in 9.2.
- Contributor
@alfredo1 Can you rebase your MR against master so we can proceed with this issue.
1 - Valery Sizov mentioned in merge request !1889 (merged)
mentioned in merge request !1889 (merged)
- Contributor
BE part with specs here https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/1889
@alfredo1 It expects
assignee_ids
orassignee_usernames
array if there are many assignees, is that possible? If not, let me know what the format would be more convenient to you.@smcgivern Could you have a quick look?
1 - Contributor
@victorwu I think we should move this to
9.3
I'm working still on https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/11098 - Maintainer
Just a heads up, the release team is having issues deploying 9.2 rc because of the multiple assignees migration. Multiple assignees might get reverted and pushed back another release
- Maintainer
@timzallmann asked me to take over this MR because @alfredo1 has limited bandwidth.
- username-removed-408677 assigned to @ClemMakesApps
assigned to @ClemMakesApps
- username-removed-408677 assigned to @victorwu and @cperessini
assigned to @victorwu and @cperessini
- Phil Hughes closed via merge request !1494 (merged)
closed via merge request !1494 (merged)
- Victor Wu reopened
reopened
- Victor Wu mentioned in issue #2972 (closed)
mentioned in issue #2972 (closed)
- Victor Wu changed title from Search for issues with multiple assignees to Search for issues with multiple assignees with implicit AND condition
changed title from Search for issues with multiple assignees to Search for issues with multiple assignees with implicit AND condition
- username-removed-408677 assigned to @vsizov
assigned to @vsizov
- username-removed-408677 unassigned @ClemMakesApps
unassigned @ClemMakesApps
- username-removed-408677 removed frontend label
removed frontend label
- username-removed-408677 mentioned in merge request !2745 (merged)
mentioned in merge request !2745 (merged)
- Valery Sizov closed
closed