Merge pull request #299 from coopdevs/error_pages

Custom error pages

app/assets/stylesheets/application.css.scss

@@ -895,3 +895,15 @@ form .checkbox input[type="checkbox"] {
     }
   }
 }
+
+.errors {
+  padding: 3em;
+
+  h3 {
+    margin-left: auto;
+    margin-right: auto;
+    max-width: 75%;
+    padding: 1em;
+    line-height: 1.6em;
+  }
+}

app/controllers/errors_controller.rb

@@ -0,0 +1,9 @@
+class ErrorsController < ApplicationController
+  def not_found
+    render status: 404
+  end
+
+  def internal_server_error
+    render status: 500
+  end
+end

app/views/errors/internal_server_error.html.erb

@@ -0,0 +1,5 @@
+<div class='text-center errors'>
+  <h1><%= t('.title') %></h1>
+  <h3><%= t('.description') %></h3>
+  <p><%= link_to t('global.home'), root_path, class: 'btn btn-default' %></p>
+</div>

app/views/errors/not_found.html.erb

@@ -0,0 +1,5 @@
+<div class='text-center errors'>
+  <h1><%= t('.title') %></h1>
+  <h3><%= t('.description') %></h3>
+  <p><%= link_to t('global.home'), root_path, class: 'btn btn-default' %></p>
+</div>

config/application.rb

@@ -25,5 +25,8 @@ class Application < Rails::Application
 
     # Do not swallow errors in after_commit/after_rollback callbacks.
     config.active_record.raise_in_transactional_callbacks = true
+
+    # This tells Rails to serve error pages from the app itself, rather than using static error pages in public/
+    config.exceptions_app = self.routes
   end
 end

config/locales/en.yml

@@ -265,6 +265,12 @@ en:
       send_paranoid_instructions: If your account exists, you will receive an email with instructions about how to unlock it in a few minutes.
       unlocked: Your account has been unlocked successfully. Please sign in to continue.
   errors:
+    internal_server_error:
+      title: Internal server error
+      description:  We're sorry, there seems to be an error with this request. A notification has automatically been sent to us, and we will resolve this as soon as possible.
+    not_found:
+      title: Not found
+      description: The page you were looking for doesn't exist. You may have mistyped the address or the page may have moved.
     messages:
       already_confirmed: was already confirmed, please try signing in
       confirmation_period_expired: needs to be confirmed within %{period}, please request a new one
@@ -289,6 +295,7 @@ en:
     edit: Update
     filter: Filter
     give_time: Time transfer
+    home: Home
     information: Information
     locales_header: change language
     member_count: 'Number of users:'

config/routes.rb

@@ -78,4 +78,6 @@
     end
   end
 
+  match '/404', to: 'errors#not_found', via: :all
+  match '/500', to: 'errors#internal_server_error', via: :all
 end