fix(alerts): memoize selecting alerts

[alex-mcgovern]

• While working on [Bug]: Duplicate requests to alerts #230 I noticed that we weren't following some best-practices around use of the select parameter for react-query's useQuery hook:

The select function will only re-run if:

• the select function itself changed referentially
• data changed
  This means that an inlined select function, as shown above, will run on every render. To avoid this, you can wrap the select function in useCallback, or extract it to a stable function reference if it doesn't have any dependencies

• This fixes any usage errors throughout the codebase (mainly created by me :D) and documents them, to attempt to avoid any regressions.

[coveralls]

Pull Request Test Coverage Report for Build 13048707362

Details

• 3 of 3 (100.0%) changed or added relevant lines in 3 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.02%) to 72.56%

---

Totals
Change from base Build 13036009495:	-0.02%
Covered Lines:	785
Relevant Lines:	1019

---

💛 - Coveralls