注文日選択を日付順にしたい about bento HOT 5 CLOSED

https://github.com/colorbox/bento/blob/53-freeze-order/app/models/order.rb#L23 のコードも Order 生成するところで休日祝日抜いてるから無くても良さげ。

from bento.

#32 (comment)

仮でいれてました。Order レコードつくるところでそこは取りますね。

from bento.

これ、 #42 で Order の存在する前提条件が変わる（土日祝日がなくなる）ので、on_weekday というインスタンスメソッドをなくして、コントローラ order#index に

def index
  @orders = Order.where('date > ?', Time.current.to_date).limit(14).order(date: :asc)
end

って書くだけにできちゃうんじゃないかなーと思っております 💭 （ where 以降が長くてちょっとな〜ってことならもちろん scope とかを定義してもいいとおもいます）

from bento.

確かにModelに寄せなくても良さそうですね！
2週間出すなら10営業日なので limit(10) の方がより厳密かなと（とはいえ、そこまで厳密にする必要もないのでおまかせですけど）

from bento.

🔚 to close

from bento.