So, seems this could be:

```ruby
if response == '200'
  add_warning('states.download_external_profile', link: transition_decorator.next_state_location,secondary_descriptor: resource_name,  
    state: state_name, transition: transition_decorator.name)
else
  add_error('states.invalid_external_location', link: transition_decorator.next_state_location,
    secondary_descriptor: resource_name, state: state_name, transition: transition_decorator.name)
end
```

Also, seems cleaner if external_document_store would return a 200 as an integer vs. a string. You might want to look at whether that is a reasonable cleanup. Then you don't have magic strings for integers.