mirror of
https://github.com/discourse/discourse.git
synced 2026-03-04 01:15:08 +08:00
23982658b2
When updating a group's notification defaults (like watching_category_ids) via API, if existing members would be affected, the server returns a 422 asking the caller to specify whether to apply changes to existing users. The previous error message was confusing: "You supplied invalid parameters to the request: update_existing_users" - even though the caller hadn't supplied that parameter at all. The new message clearly explains what's happening and what to do: "This change affects X existing group members. You must specify the 'update_existing_users' parameter (true or false) to indicate whether to apply the new notification defaults to existing members." Also fixed a bug where the confirmation modal state wasn't being reset when navigating between groups, causing the modal to not appear on subsequent group edits. The modal now returns its result directly instead of storing it in component state. Ref - https://meta.discourse.org/t/-/393572 |
||
|---|---|---|
| .. | ||
| group_members_spec.rb | ||
| group_spec.rb | ||
