Not too bad, but you have a few problems: In breadboard the pins are not on .1 centers (that is OK if in real life they also aren't on .1 centers, but that seems less than likely). Schematic you should eliminate the nc (which I'm assuming is no connection) from the pins and probably label the signal connection as signal. You also need to set a connectorxterminal in the schematic svg to place the connection point on the end of the pin. The definition is in the fpz file, but isn't present in the schematic.svg and thus the wires connect to the middle of the pin (which isn't correct). I usually use a .01 by .01 rectangle with fill=none as a terminal.
Peter