parabol icon indicating copy to clipboard operation
parabol copied to clipboard

Check-in: Task assignee menu is missing entries

Open Dschoordsch opened this issue 2 years ago • 4 comments

How to reproduce

  • Start check-in meeting with 3 participants
  • Try reassigning a task in Solo updates
  • Try reassigning a task in Agenda item discussion

Expected behaviour

Assignee menu shows all team members.

Actual behaviour

Assignee menu shows different subset of members depending on stage. Screenshot from 2022-07-07 13-37-03 Screenshot from 2022-07-07 13-34-34

Dschoordsch avatar Jul 07 '22 12:07 Dschoordsch

@Dschoordsch what's the desired behavior here when the current assignee is selected from the list of team members? No-op, make task unassigned, something else?

jmtaber129 avatar Sep 21 '22 00:09 jmtaber129

@Dschoordsch what's the desired behavior here when the current assignee is selected from the list of team members? No-op, make task unassigned, something else?

We don't have unassigned tasks, so a no-op would be fine. We could also filter out the author, but then the list is different for each tasks and if you assign multiple tasks it could be annoying. I would prefer to show all members but disable the entry for the current assignee.

Dschoordsch avatar Sep 22 '22 07:09 Dschoordsch

@Dschoordsch I tried to repro this on master to compare master to https://github.com/ParabolInc/parabol/pull/7160, but I'm not actually able to repro unless I'm the only one on the team.

Can you provide a bit more detail about how to repro?

jmtaber129 avatar Sep 22 '22 18:09 jmtaber129

@jmtaber129 I don't have more reproduction information, we noticed this when running a check-in as release smoke test with 3 participants.

Dschoordsch avatar Sep 23 '22 14:09 Dschoordsch

Closing this as we don't have a repro

jmtaber129 avatar Mar 13 '23 18:03 jmtaber129