Some improvements to navigation, and pulling out separate show/edit views for customers.

8 files changed, 24 insertions, 20 deletions

diff --git a/billing/app/views/credit_card_info/edit.html.haml b/billing/app/views/credit_card_info/edit.html.haml
index 95f450f..57322de 100644
--- a/billing/app/views/credit_card_info/edit.html.haml
+++ b/billing/app/views/credit_card_info/edit.html.haml
@@ -13,4 +13,5 @@
       %dt= f.label :cvv, 'CVV'
       %dd= f.text_field :cvv
   = hidden_field_tag :tr_data, @tr_data
-  = f.submit 'Save Payment Info'
\ No newline at end of file
+  = f.submit 'Save Payment Info', :class => :btn
+  = link_to t(:cancel), edit_customer_path(@credit_card.customer_id), :class => :btn
\ No newline at end of file
diff --git a/billing/app/views/customer/_transaction.html.haml b/billing/app/views/customer/_transaction.html.haml
index 97842dc..53483d9 100644
--- a/billing/app/views/customer/_transaction.html.haml
+++ b/billing/app/views/customer/_transaction.html.haml
@@ -2,7 +2,7 @@
   = transaction.id
   Type:
   = transaction.type
-  Transaction Amount:
+  Amount:
   = number_to_currency(transaction.amount)
   Status:
   = transaction.status
diff --git a/billing/app/views/customer/edit.html.haml b/billing/app/views/customer/edit.html.haml
index 2104b09..bea3211 100644
--- a/billing/app/views/customer/edit.html.haml
+++ b/billing/app/views/customer/edit.html.haml
@@ -11,19 +11,12 @@
       %dd= f.text_field :last_name
       %dt= f.label :phone, 'Phone'
       %dd= f.text_field :phone
-  - if @default_cc
-    %ul
-      %li
-        Stored Credit Card
+      - if @default_cc
+        = # todo, as they will need a credit card, so not sure about conditional?
+        %dt Stored Credit Card
+        %dd
         = @default_cc.masked_number
-        = link_to 'Change credit card', edit_credit_card_info_path(:id => @default_cc.token)
-      %li
-        Transaction History
-        = render(:partial => "transaction", :collection => @transactions)
-      - if @subscriptions.any?
-        %li
-          Active Subscriptions
-          = # todo: won't really have multiple subscriptions
-          = render(:partial => "subscription", :collection => @subscriptions)
+        = link_to 'Change credit card', edit_credit_card_info_path(:id => @default_cc.token), :class => :btn
   = hidden_field_tag :tr_data, @tr_data
-  = f.submit 'Save Customer Info'
\ No newline at end of file
+  = f.submit 'Save Customer Info', :class => :btn
+= link_to 'Cancel', customer_path(@braintree_data.id), :class=> :btn
\ No newline at end of file
diff --git a/billing/app/views/customer/show.html.haml b/billing/app/views/customer/show.html.haml
new file mode 100644
index 0000000..7416682
--- /dev/null
+++ b/billing/app/views/customer/show.html.haml
@@ -0,0 +1,8 @@
+= render :partial => 'payments/customer_data'
+= link_to 'Make Payment', new_payment_path, :class => :btn
+%h3 Transaction History
+= render(:partial => "transaction", :collection => @transactions) # show subset with link to see more
+- if @subscriptions.any?
+  %h3 Active Subscriptions
+  = # todo: won't really have multiple subscriptions
+  = render(:partial => "subscription", :collection => @subscriptions)
\ No newline at end of file
diff --git a/billing/app/views/payments/_customer_data.html.haml b/billing/app/views/payments/_customer_data.html.haml
index 900fbd8..d46065c 100644
--- a/billing/app/views/payments/_customer_data.html.haml
+++ b/billing/app/views/payments/_customer_data.html.haml
@@ -12,4 +12,4 @@
   %dd= @default_cc.masked_number
   %dt Expiration Date
   %dd= @default_cc.expiration_date
-= link_to 'edit saved data', edit_customer_path(@customer.braintree_customer_id), :class => :btn
+= link_to 'Edit Saved Data', edit_customer_path(@customer.braintree_customer_id), :class => :btn
diff --git a/billing/app/views/payments/new.html.haml b/billing/app/views/payments/new.html.haml
index 1d958b9..d1d0aa9 100644
--- a/billing/app/views/payments/new.html.haml
+++ b/billing/app/views/payments/new.html.haml
@@ -12,4 +12,4 @@
   - else
     = render :partial => 'customer_data'
   = hidden_field_tag :tr_data, @tr_data
-  = f.submit "Submit"
\ No newline at end of file
+  = f.submit "Submit Payment", :class => :btn
\ No newline at end of file
diff --git a/billing/app/views/subscriptions/_subscription_details.html.haml b/billing/app/views/subscriptions/_subscription_details.html.haml
index db07204..db9d75b 100644
--- a/billing/app/views/subscriptions/_subscription_details.html.haml
+++ b/billing/app/views/subscriptions/_subscription_details.html.haml
@@ -1,5 +1,5 @@
 %p
-  = subscription.id
+  = link_to subscription.id, subscription_path(subscription.id)
   Balance:
   = number_to_currency(subscription.balance)
   Bill on:
diff --git a/billing/app/views/subscriptions/show.html.haml b/billing/app/views/subscriptions/show.html.haml
index bc85fd9..b630a05 100644
--- a/billing/app/views/subscriptions/show.html.haml
+++ b/billing/app/views/subscriptions/show.html.haml
@@ -1,2 +1,4 @@
+%h1 Current Subscription
 = render :partial => "subscription_details",  :locals => {:subscription => @subscription}
-= link_to t(:Cancel), subscription_path,  :confirm => 'Are you sure?', :method => :delete, :class => 'btn btn-danger' if @subscription.status == 'Active' # permission check or should that just be on show?
\ No newline at end of file
+= link_to 'Cancel Subscription', subscription_path,  :confirm => 'Are you sure you want to cancel this subscription?', :method => :delete, :class => 'btn btn-danger' if @subscription.status == 'Active' # permission check or should that just be on show?
+= link_to 'Show Customer Data', customer_path(@subscription_customer_id), :class => :btn
\ No newline at end of file