dhickie
dhickie
I've spent some time digging into this. When looking at the `DepositPost(Async)` method, there are two overloads - one that accepts an implementation of `IAmABoxTransactionProvider` as an argument, and one...
Thanks Ian, that clarifies the difference between `Post` and `DepositPost` for me a lot. I think it's reasonable for users to expect to be able to use `Post` when they...
@iancooper This is fixed in the main branch.
@iancooper Happy to take a look at this one. Is one of `release/9x` or `release/9X` the branch I should look at for forking?
The issue is that if you have a dictionary of resource IDs to lease IDs, then when Run B begins and successfully obtains a lease, then by the time Run...
@preardon Looks good to me 👍
Yeah I thought about the same thing. I agree that implicit meaning from `null` values isn't the best (though historically fairly common). I don't tend to see tuples getting returned...
@iancooper I could take a look at this if you're busy elsewhere? I'd just need a bit of context around the intent behind requiring a single topic name when fetching...
It looks like this is something specific to the Dynamo DB outbox implementation. I can see why it's there - the GSI for dispatched messages uses the topic name as...
@iancooper > Interestingly in V10 we have surfaced RequestContext on most calls, we could actually pass that information there and drop args . I didn't do that because I would...