plans and products

app/controllers/admin/plans_controller.rb

@@ -20,13 +20,11 @@ module DiscoursePatrons
 
       def create
         begin
-
           plan = ::Stripe::Plan.create(
             amount: params[:amount],
             interval: params[:interval],
-            product: { name: params[:name] },
+            product: product,
             currency: SiteSetting.discourse_patrons_currency,
-            id: plan_id,
           )
 
           render_json_dump plan
@@ -49,8 +47,8 @@ module DiscoursePatrons
 
       private
 
-      def plan_id
-        params[:name].parameterize.dasherize if params[:name]
+      def product
+        params[:product].slice(:id, :name).permit!.to_h.symbolize_keys if params[:product]
       end
     end
   end

app/controllers/admin/products_controller.rb

@@ -47,11 +47,10 @@ module DiscoursePatrons
       def update
         begin
           product = ::Stripe::Product.update(
-            params[:id], {
-              name: params[:name],
-              active: params[:active],
-              metadata: metadata
-            }
+            params[:id],
+            name: params[:name],
+            active: params[:active],
+            metadata: metadata
           )
 
           render_json_dump product

app/controllers/plans_controller.rb

@@ -7,8 +7,14 @@ module DiscoursePatrons
     before_action :set_api_key
 
     def index
-      plans = ::Stripe::Plan.list
-      render json: plans.data
+      begin
+        plans = ::Stripe::Plan.list
+
+        render_json_dump plans.data
+
+      rescue ::Stripe::InvalidRequestError => e
+        return render_json_error e.message
+      end
     end
   end
 end

app/controllers/subscriptions_controller.rb

@@ -7,14 +7,31 @@ module DiscoursePatrons
     before_action :set_api_key
 
     def create
-      subscription = ::Stripe::Subscription.create(
-        customer: params[:customer],
-        items: [
-          { plan: params[:plan] },
-        ]
-      )
+      begin
+        subscription = ::Stripe::Subscription.create(
+          customer: params[:customer],
+          items: [
+            { plan: params[:plan] },
+          ]
+        )
 
-      render_json_dump subscription
+        if subscription_ok(subscription)
+          # TODO: check group credentials
+          group = Group.find_by_name('group-123')
+          group.add(current_user)
+        end
+
+        render_json_dump subscription
+
+      rescue ::Stripe::InvalidRequestError => e
+        return render_json_error e.message
+      end
+    end
+
+    private
+
+    def subscription_ok(subscription)
+      ['active', 'trialing'].include?(subscription[:status])
     end
   end
 end