Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[#925] Disable ReserveStock and Bid Variation Backorders #932

Merged
merged 2 commits into from
Apr 29, 2024

Conversation

bd-viget
Copy link
Contributor

Summary

This PR modifies the Bid Variation behavior by disallowing backorders and disabling ReserveStock to allow multiple users place the same bid item in their cart. This still needs testing to make sure 2 users cannot order the same Bid Product at the same time, but hopefully the other mechanisms (bid locking, quantity reduction/checking) prevent that.

Issues

Testing Instructions

  1. Try to bid on the same Auction at the same time for the same bid with 2 different users.
  2. One user shouldn't be able to complete their order.

@bd-viget bd-viget requested a review from clatwell April 26, 2024 20:23
@bd-viget bd-viget self-assigned this Apr 26, 2024
@bd-viget bd-viget requested a review from jaspercroome April 29, 2024 13:18
@bd-viget bd-viget merged commit ffe89b1 into main Apr 29, 2024
1 check passed
@bd-viget bd-viget deleted the bd/925-duplicate-bid-orders branch April 29, 2024 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants