i'm not so keen on duplicating part of the logic that's in `is_translatable_value`. i agree that `value == PLACEHOLDER or not is_translatable_value(value)` is ugly, but at least it's dry

maybe we want a constant for the tuple `(None, "") or something?