Skip to content

Commit

Permalink
# Avoiding checking for default selected option (not all subsets have…
Browse files Browse the repository at this point in the history
… a default)
  • Loading branch information
maurok committed May 17, 2017
1 parent f5179c3 commit c5cf67b
Showing 1 changed file with 5 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -148,10 +148,11 @@ private async Task OnInvoke(IInvokeActivity invoke, IConnectorClient connectorCl
}
}

if (result.Details.ShippingOptions.Count(option => option.Selected.HasValue && option.Selected.Value) != 1)
{
throw new ArgumentException("Expected exactly zero or one selected shipping option.");
}
// do not require or check default selected option
//if (result.Details.ShippingOptions.Count(option => option.Selected.HasValue && option.Selected.Value) != 1)
//{
// throw new ArgumentException("Expected exactly zero or one selected shipping option.");
//}

// update payment details after shipping changed
await this.shippingService.UpdatePaymentDetailsAsync(result.Details, paymentRequestUpdate.ShippingAddress, catalogItem);
Expand Down

0 comments on commit c5cf67b

Please sign in to comment.