mirror of
https://github.com/discourse/discourse.git
synced 2026-08-07 13:19:19 +08:00
0f5ab6e940Previously, picking a group in the bulk assign modal always failed with a bodyless 500: the client only sent `username`, `group_name` was not a permitted bulk action parameter, and the bulk operation only ever resolved a `User` — so `Assigner` received `nil`, treated it as a `Group`, and crashed dereferencing it. This change routes both the bulk and single-topic paths through one `DiscourseAssign::AssigneeResolver`, makes `Assigner#assign` return a reason instead of raising for caller-supplied input, and surfaces per-topic refusals through `@errors` so a partially applied bulk assign explains itself instead of reporting success. Ref - t/188206 |
||
|---|---|---|
| .. | ||
| discourse_assign | ||
| discourse_workflows/nodes | ||
| assigner_spec.rb | ||
| pending_assigns_reminder_spec.rb | ||
| random_assign_utils_spec.rb | ||
| topic_query_spec.rb | ||
