Convert status checking to include all good statuses

[Timothy-Gonzalez]

Right now, you guys are checking explicitly for status code to be 200. There are other status codes, that while less common, are still "good" statuses. For example, SuccessCreated is 201, SuccessNoContent is 204.

The proper check is res.statusCode < 200 || res.statusCode >= 300. I think there's also some library call to do this automatically for you.

I could only find this check in 3 places, but it might elsewhere:

• site/utils/api.ts

  Line 48 in c717b0d

  if (response.status !== 200) {

• site/utils/api.ts

  Line 75 in c717b0d

  if (response.status !== 200) {

• site/utils/api.ts

  Line 135 in c717b0d

  if (response.status !== 200) {