IntenseWebs/discourse
3
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2025-02-25 18:55:32 -06:00
Code Issues Packages Projects Releases Wiki Activity

FIX: don't send multiple requests when changing category notification (#6435)

Browse Source
This commit is contained in:
Maja Komel 2018-10-01 17:41:23 +02:00 committed by Régis Hanol
parent e2770bc1c4
commit 1add11d8c9
2 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
app/assets/javascripts/select-kit/components/category-notifications-button.js.es6
View File

@ -11,5 +11,7 @@ export default NotificationOptionsComponent.extend({
mutateValue(value) { mutateValue(value) {
this.get("category").setNotification(value); this.get("category").setNotification(value);
} },
deselect() {}
}); });

6
app/models/category_user.rb
View File

@ -55,7 +55,11 @@ class CategoryUser < ActiveRecord::Base
record.notification_level = level record.notification_level = level
record.save! record.save!
else else
CategoryUser.create!(user: user, category_id: category_id, notification_level: level) begin
CategoryUser.create!(user: user, category_id: category_id, notification_level: level)
rescue ActiveRecord::RecordNotUnique
# does not matter
end
end end
auto_watch(user_id: user.id) auto_watch(user_id: user.id)