FE - Monitor Sort Order

[ajhollid]

Currently the monitors cannot be sorted:

If we want to enable we can sort on either the client or the backend, with the usual tradeoffs.

Client side

• Downside: Potentially slow if many monitors
• Upside: No need for API call, easy to implement

Server side

• Downside: Requires an API call, marginally more complicated to implement
• Upside: Client will not be blocked while waiting for data

[Skorpios604]

Seems like a good idea. Some people might want to sort by which monitors are up or down. Like where that arrow is next to status. Users might intuitively click that thinking that it will sort the monitors for them. It is a very wide spread feature.

[gorkem-bwl]

My take: let's go with the client side and if things got slow with hundreds of servers, we can easily switch (I hope that happens lol). Eventually we'll add filters on top of the table, helping users to view only the up or down monitors

[gorkem-bwl]

Another take: The incidents page consists of more than 2500 entries and client side filtering seems to be fast enough. I am not in a position to compare the amount of data on the table (dashboard) and the table (incidents) but I tend to believe it shouldn't be an issue to go client side here.

[ajhollid]

Alright! Let's do it client side and see how it works, always possible to change later if performance degrades.