shouldn't this check blank to? unless transition_url.blank? If we warn blank transitions elsehere, do you think this is redundant to warn here? Seems these warnings should be classes so that in these serializers, there are no implementation details like warnings. That is all warnings should occur somewhere upstream in the representor interface and classes.