Yes, that is a proper interface from opto to relay switching transistor. I would not bother with having two visual leds performing the same function, I would eliminate the opto input led indicator and adjust it's series resistor to compensate for it's elimination.
Lefty