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

Set lot winners screen should be more responsive #159

Closed
iragm opened this issue Jun 8, 2024 · 4 comments
Closed

Set lot winners screen should be more responsive #159

iragm opened this issue Jun 8, 2024 · 4 comments

Comments

@iragm
Copy link
Owner

iragm commented Jun 8, 2024

The following changes would make the set lot winners screen better:

  • A button to go into full screen mode (requires a div for the form and all related stuff so the page doesn't reload)
  • AJAX update field statuses with issues like lot not found, user invoice closed, etc.
@iragm
Copy link
Owner Author

iragm commented Aug 16, 2024

If the lot has a reserve price set, there should be some kind of friction if you try to sell the lot for less than the reserve.
From #181:

This should launch a prompt (reserve price is $x, are you sure you want to sell for $y).

@iragm
Copy link
Owner Author

iragm commented Aug 29, 2024

If the auction is in-person and bidding is allowed, add in information discussed #189 (comment)

@iragm
Copy link
Owner Author

iragm commented Sep 11, 2024

Should be pretty easy to post a message in real time to the lot page when a lot is going up for auction (view_lot_simple) and when it's sold, too

@iragm
Copy link
Owner Author

iragm commented Sep 16, 2024

This project spiraled as things tend to:

  • Lot page now updates in real time as winners get set
  • When setting lot winners, there's a new button to ignore errors and save
  • Users that watch items in an in-person auction now have the ability to get push notifications when bidding starts on a lot
  • Invoices are recalculated immediately instead of a few seconds later (previously this was done in the task that ends auctions)
  • A per-user websocket connection is now available on all pages, although it is currently disabled, it may be handy for future improvements

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

No branches or pull requests

1 participant