added parameter for max_alerts

TOMToolkit / tom_antares

Antares Broker module for the TOM Toolkit

GNU General Public License v3.0

0 stars 5 forks source link

added parameter for max_alerts #29

Closed brendanrmills closed 1 year ago

brendanrmills commented 2 years ago

I added a parameter to the fetch_alerts function that allows a user to change the number of alerts returned. Before, the maximum number of alerts was hard coded as 20, but changing that to be part of the query provides users and developers more flexibility to query for longer more general sets of alerts. I reflected this change in the broker query and testing suite as well.

rachel3834 commented 2 years ago

Added Joey and Lindy to request review

jchate6 commented 2 years ago

Same Token requirements for the coverage test as the other PR. Think we are stuck here until @phycodurus returns as well.