fixed request type strings to match database

[jmensch1]

Fixes #428

The main change here is to the shape of the REQUEST_TYPES constant. As we discussed I turned it into an object, which let me remove a bunch of .find() statements.

// previously
export const REQUEST_TYPES = [
  {
    type: 'Dead Animal',
    fullType: 'Dead Animal Removal',
    abbrev: 'DAN',
    color: '#4FEFEF',
  }
...
]

// now
export const REQUEST_TYPES = {
  'Dead Animal Removal': {           // matches the database string
    displayName: 'Dead Animal',    // for UI purposes
    abbrev: 'DAN',
    color: '#4FEFEF',
  },
...
}

Every type has a displayName, so you don't have to worry about testing for a fallback.

I also removed a couple of constants that weren't being used anywhere.

[adamkendis]

Nice, this is way better.