Hi Paul
Glad to hear you got your synch working an the Integration Framework complete
Reason why I had Logical System = Connector: I used the naming convention <SID>CLNT<Client> (eg PRDCLNT100) for my RFC Connection in SM59. I then used the same name for the BD54 Logical System and my Connector. This way it was easier to track my config through the system and easily identify the connection
Another bit I get caught out with the synch: the connector value is case sensitive. If you do not enter the connector in capitals (or however defined) you will get error message "Connector is not properly configured in common component settings"