Paul Reardon
Paul Reardon
@iancooper let me know if you're ok for me to go ahead with this, Also would you like it to be DepositPost or would you like it to stick out...
@iancooper not suggesting a breaking change, just an overload, happy to do it either way. I was thinking that we may want a new Interface for IAmABulkOutboxAsync
This has been delivered is 9.2.3
``` System.Data.SqlTypes.SqlNullValueException: at Microsoft.Data.SqlClient.SqlBuffer.ThrowIfNull (Microsoft.Data.SqlClient, Version=4.1.0.0, Culture=neutral, PublicKeyToken=23ec7fc2d6eaa4a5) at Microsoft.Data.SqlClient.SqlBuffer.get_String (Microsoft.Data.SqlClient, Version=4.1.0.0, Culture=neutral, PublicKeyToken=23ec7fc2d6eaa4a5) at Microsoft.Data.SqlClient.SqlDataReader.GetString (Microsoft.Data.SqlClient, Version=4.1.0.0, Culture=neutral, PublicKeyToken=23ec7fc2d6eaa4a5) at Paramore.Brighter.Outbox.MsSql.MsSqlOutbox.MapAMessage (Paramore.Brighter.Outbox.MsSql, Version=9.0.0.0, Culture=neutral, PublicKeyToken=b90f5623ae237e2c: /_/src/Paramore.Brighter.Outbox.MsSql/MsSqlOutbox.cs:704) at Paramore.Brighter.Outbox.MsSql.MsSqlOutbox+d__23.MoveNext (Paramore.Brighter.Outbox.MsSql,...
The fix has been released in 9.2.4
Looks good to me
It's specifically marked as private as it isn't considered as something we should be parsing in / a concern of your application. However this approach doesn't allow multiple dB contexts....
So the Problem with option two is you have then tied the command processor to adodb Transaction. I was thinking of something more towards a Transaction provider that takes a...
If we inject the IAmABoxTransactionProvider this would work, however that is a Brighter internal and I feel it's better to set a Default and give the ability to Pass a...
@IlSocio doing what you've stated above would close this to other providers as well as bring along additional dependencies, I will knock out a quick Draft PR of something that...