Skip to content

Commit 8a57009

Browse files
authored
Merge pull request #202 from coderdojo-japan/dojo-order
Dojo order
2 parents a75d348 + c9eaac8 commit 8a57009

File tree

3 files changed

+4
-3
lines changed

3 files changed

+4
-3
lines changed

app/controllers/plain_page_controller.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,6 @@ class PlainPageController < CmsController
22
#skip_before_action :verify_authenticity_token, only: [:index]
33

44
def index
5-
@dojos = Dojo.all
5+
@dojos = Dojo.default_order.all
66
end
77
end

app/controllers/static_pages_controller.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
class StaticPagesController < ApplicationController
22
def home
33
@dojo_count = Dojo.count
4-
@regions_and_dojos = Dojo.includes(:prefecture).group_by { |dojo| dojo.prefecture.region }
4+
@regions_and_dojos = Dojo.includes(:prefecture).default_order.group_by { |dojo| dojo.prefecture.region }
55
end
66

77
def letsencrypt

app/models/dojo.rb

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,10 @@ class Dojo < ApplicationRecord
99
has_many :event_histories, dependent: :destroy
1010

1111
serialize :tags
12-
default_scope -> { order(order: :asc) }
1312
before_save { self.email = self.email.downcase }
1413

14+
scope :default_order, -> { order(prefecture_id: :asc) }
15+
1516
validates :name, presence: true, length: { maximum: 50 }
1617
validates :email, presence: false
1718
validates :order, presence: false

0 commit comments

Comments
 (0)