Closed da-ar closed 5 years ago
+1
@clairecadman can you have a quick look over the changes here, and apply your magic, please?
@DavidS Yes I'll take a look this afternoon!
@DavidS I made a couple of minor edits. But line 453 needs to be fixed, I'm just not sure what it means. Thanks!
@DavidS reminder that you have some local changes to push.
Not happy with the transport implementation being described here, I think we need an interface class.
Added file paths to the code samples, as in @DavidS notes. I think they help make things clearer.