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

Manualni zavirani aukci #46

Open
wants to merge 21 commits into
base: manual-closing
Choose a base branch
from
Open

Manualni zavirani aukci #46

wants to merge 21 commits into from

Conversation

mreq
Copy link
Member

@mreq mreq commented Mar 8, 2022

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.

@mreq mreq requested review from foton, ornsteinfilip and dedekm March 8, 2022 10:29
Gemfile Show resolved Hide resolved
@@ -249,6 +263,23 @@ def auction_prolonging_limit_in_seconds
pack&.auction_prolonging_limit_in_seconds || Auctify.configuration.auction_prolonging_limit_in_seconds
end

def close_manually(by:, price_check:)
if manually_closed_at.nil? && update(manually_closed_at: Time.current, manually_closed_by: by, manually_closed_price_check: price_check)
Auctify::BiddingCloserJob.perform_later(auction_id: id)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tady mi přijde zbytečné to honit přes Job. Rovnou bych dal auction.close_bidding!

end
end

def lock_bidding(by:)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tady budu potřebovat objasnit jaký je rozdíl/použití auction.lick_bidding a auction.bidding_closed. Respektive kdy dochází k unlock_bidding

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lock_bidding a unlock_bidding jsou adminem rucne volane akce z frontendu, ktere slouzi pro mezikrok kontroly dosazene ceny oproti salu pred samotnym manualnim uzavrenim

image

image

@foton
Copy link
Collaborator

foton commented Mar 17, 2022

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
Copy link
Member Author

mreq commented Mar 17, 2022

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.

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