sinfin / auctify

Rails engine for auctions of items
MIT License
11 stars 4 forks source link

Manualni zavirani aukci #46

Open mreq opened 2 years ago

mreq commented 2 years ago

PR pro celkovy prehled provedenych zmen.

Sale muze mit must_be_closed_manually, coz meni jeho chovani.

Nemiri to do master, PR je tu proto, aby si ho mohli @dedekm a @ornsteinfilip precist.

foton commented 2 years ago

Poznámky k diskusi: Po auction.close_bidding! se zablokují další příhozy a nastaví vítěz a čas ukončení. Taky se volá after_close_bidding a process_bidding_result! if configuration.autofinish_auction_after_bidding == true. Pro ruční ukončování všech aukcí by tedy mělo stačit: 0) configuration.autofinish_auction_after_bidding = false v config/initializers/auctify.rb (je to default) 1) pro ukončení přihazování volat auction.close_bidding 2) umožnit připadnou editaci příhozů 3) pak znovu nastavit vítěze 4) a zavolat process_bidding_result

Respektive, pokud se přiřazení vítěze přesune do process_bidding_result tak to bude ještě lepší. U automaticky zpracovaných aukcí je to jedno a u ručních bude jen potřeba editovat příhozy před spuštění process_bidding_result

mreq commented 2 years ago

Poznámky k diskusi: Po auction.close_bidding! se zablokují další příhozy a nastaví vítěz a čas ukončení. Taky se volá after_close_bidding a process_bidding_result! if configuration.autofinish_auction_after_bidding == true. Pro ruční ukončování všech aukcí by tedy mělo stačit: 0) configuration.autofinish_auction_after_bidding = false v config/initializers/auctify.rb (je to default)

  1. pro ukončení přihazování volat auction.close_bidding
  2. umožnit připadnou editaci příhozů
  3. pak znovu nastavit vítěze
  4. a zavolat process_bidding_result

Respektive, pokud se přiřazení vítěze přesune do process_bidding_result tak to bude ještě lepší. U automaticky zpracovaných aukcí je to jedno a u ručních bude jen potřeba editovat příhozy před spuštění process_bidding_result

Pak si na to zavolejme. Ted uz bych to samozrejme nepredelaval, jestli jsi nenasel nejaky zadrhel.