Duplicate request when changing notification level

(Blake Erickson) #1

When you want to change your notification level for a topic, ember is sending duplicate api requests.

(Sam Saffron) #3

I bet this is happening cause the message bus is sending it a message telling it that the notification level changed and that triggers a second refresh, worth fixing for sure.

(Sam Saffron) #4

fixed per:

A lot of history behind this code, it still feels a bit messy, last time around @tgxworld needed to fix it so keyboard shortcuts update the box, a side effect was the double ajax calls

(Sam Saffron) closed #5

(Alan Tan) #6

Thanks for fixing. I was actually fixing the case where changing the notification level using the component in the topic menu item wasn’t updating the notification level in the widget mounted within the timeline. Anyway I had a look into the code path again and realized that I should have just re-rendered the topic timeline again.