opserver / Opserver

Stack Exchange's Monitoring System
https://opserver.github.io/Opserver/
MIT License
4.51k stars 827 forks source link

Loading more exceptions doesn't respect current exceptions group #273

Closed AlexSikilinda closed 7 years ago

AlexSikilinda commented 7 years ago

Scenario: I select an exception group, let's say "DEV". I scroll to the bottom of the page thus triggering exceptions/load-more API. The problem is that this API call returns exceptions from all groups, not only "DEV".

This happens because we do not pass group parameter from JS: image

The solution is to add group: options.group to javascript object. Can submit a pull request but wonder if it was done intentionally.

NickCraver commented 7 years ago

Nope just missed it - if you submit a PR I'd love to have your name in the commit list :) Thanks!