Minor change to ICP API v2 result set

  • 99 Views
  • Last post 3 weeks ago
Phil Bishop posted this 3 weeks ago

Until today, the ICP API would return records for meter components that have been flagged as removed, yet the date field showing when the meter was actually removed was not in the result set. This had the potential to confuse API users into believing the meter remained installed and active when it was neither.

Such records are now being filtered out of the result set; all meter component records can reliably be assumed to pertain to currently active meters.

Cheers

Phil

Order by: Standard | Newest | Votes
Clive Gifford posted this 3 weeks ago

Interesting. What happens if an installation has no meters? I'm sure the EA tested that case, right? Or maybe the users of the API are meant to do the testing?

 


0001060300WMD10
0001111171WM17A

{"Code":500,"Message":"API is currently unavailable to process this request. (R)"}


Relevant:

"One small change for you, one giant bug for production."
"It worked before I fixed it."
"The road to downtime is paved with unchecked commits."
"If it ain't tested, it ain't done."
"A five-minute fix can cost five days in QA."
"‘Just a tiny tweak’ – famous last words."
"Code changes are like surgeries: even minor ones can go horribly wrong."
...

Guy Ross posted this 3 weeks ago

Cheers Clive,

Fixed now.

 

"Testing can be used to show the presence of bugs, but not to show their absence." - Edsger Dijkstra 
"If you're not failing, you're not trying hard enough," - Martin Fowler 
"Failure is success in progress," - Albert Einstein