order the payments in admin

app/controllers/admin_controller.rb

@@ -3,9 +3,19 @@
 module DiscoursePatrons
   class AdminController < ::Admin::AdminController
     def index
-      payments = Payment.all
+      payments = Payment.all.order(payments_order)
 
       render_serialized(payments, PaymentSerializer)
     end
+
+    private
+
+    def payments_order
+      if %w(created_at amount).include?(params[:order])
+        params[:order].to_sym
+      else
+        :created_at
+      end
+    end
   end
 end

assets/javascripts/discourse/controllers/admin-plugins-discourse-patrons.js.es6

@@ -1,5 +1,12 @@
 export default Ember.Controller.extend({
+  queryParams: ["order"],
+  order: null,
+
   actions: {
-    loadMore() {}
+    loadMore() {},
+
+    orderPayments(order) {
+      this.set("order", order);
+    }
   }
 });

assets/javascripts/discourse/routes/admin-plugins-discourse-patrons.js.es6

@@ -1,9 +1,16 @@
 import { ajax } from "discourse/lib/ajax";
 
 export default Discourse.Route.extend({
-  model() {
+  queryParams: {
+    order: {
+      refreshModel: true
+    }
+  },
+
+  model(params) {
     return ajax("/patrons/admin", {
-      method: "get"
+      method: "get",
+      data: { order: params.order }
     }).then(results => results);
   }
 });

assets/javascripts/discourse/templates/admin/plugins-discourse-patrons.hbs

@@ -9,8 +9,8 @@
           <th>{{i18n 'discourse_patrons.admin.table.head.user'}}</th>
           <th>{{i18n 'discourse_patrons.admin.table.head.payment_intent'}}</th>
           <th>{{i18n 'discourse_patrons.admin.table.head.receipt_email'}}</th>
-          <th>{{i18n 'created'}}</th>
-          <th class="amount">{{i18n 'discourse_patrons.admin.table.head.amount'}}</th>
+          <th onclick={{action "orderPayments" "created_at"}} class="sortable">{{i18n 'created'}}</th>
+          <th class="amount" onclick={{action "orderPayments" "amount"}} class="sortable amount">{{i18n 'discourse_patrons.admin.table.head.amount'}}</th>
         </tr>
       </thead>
       {{#each model as |payment|}}

spec/controllers/discourse_patrons/admin_controller_spec.rb

@@ -9,8 +9,5 @@ module DiscoursePatrons
     it 'is a subclass of AdminController' do
       expect(DiscoursePatrons::AdminController < Admin::AdminController).to eq(true)
     end
-
-    # TODO authenticate
-    it 'has payments'
   end
 end