OK, this looks fine, it seems standard 24VAC sprinkler valve and should work well with DC-powered OpenSprinkler. Regarding the sensor port: you don’t need to short the sensor port on OpenSprinkler — the sensor on OpenSprinkler is configurable in software. So if you haven’t selected any sensor in software, it won’t read the sensor terminal at all.
Just to make sure the wiring is correct, here is what I recommend you do:
remove power from OpenSprinkler, then use a multimeter to measure the resistance between the COM (common) terminal (there are two COM ports, you can use either since they are internally wired together) and each individual zone wire. The resistance should be around 20 to 50 ohm. If it’s too small or too large, that indicates a problem with either the wiring or the solenoid.
Did your OpenSprinkler 2.3 come with a 7.5VDC power adapter? If so, I recommend using that, before using your 12VDC power adapter. 7.5VDC provides ideal holding current (about 250mA according to your valve’s spec). Using 12VDC generates more holding current than necessary, which is fine but it’s less power efficient.