Option to only show disrupted days. Closes #2088

cachethq · Oct 4, 2016 · 179de1f · 179de1f
1 parent 0646b6b
commit 179de1f
Show file tree

Hide file tree

Showing 5 changed files with 30 additions and 4 deletions.
diff --git a/app/Composers/AppComposer.php b/app/Composers/AppComposer.php
@@ -78,5 +78,6 @@ public function compose(View $view)
         $view->withTimezone($this->dates->getTimezone());
         $view->withSiteTitle($this->config->get('setting.app_name'));
         $view->withFontSubset($this->config->get('langs.'.$this->config->get('app.locale').'.subset', 'latin'));
+        $view->withOnlyDisruptedDays($this->config->get('setting.only_disrupted_days'));
     }
 }
diff --git a/app/Http/Controllers/StatusPageController.php b/app/Http/Controllers/StatusPageController.php
@@ -90,11 +90,13 @@ public function showIndex()
         });
 
         // Add in days that have no incidents
-        foreach ($incidentDays as $i) {
-            $date = app(DateFactory::class)->make($startDate)->subDays($i);
+        if (Config::get('setting.only_disrupted_days') === false) {
+            foreach ($incidentDays as $i) {
+                $date = app(DateFactory::class)->make($startDate)->subDays($i);
 
-            if (!isset($allIncidents[$date->toDateString()])) {
-                $allIncidents[$date->toDateString()] = [];
+                if (!isset($allIncidents[$date->toDateString()])) {
+                    $allIncidents[$date->toDateString()] = [];
+                }
             }
         }
 

diff --git a/config/setting.php b/config/setting.php
@@ -88,4 +88,15 @@
     */
 
     'skip_subscriber_verification' => false,
+
+   /*
+    |--------------------------------------------------------------------------
+    | Only disrupted days
+    |--------------------------------------------------------------------------
+    |
+    | Whether to only show days with incidents, or each day in the timeline.
+    |
+    */
+
+    'only_disrupted_days' => false,
 ];
diff --git a/resources/lang/en/forms.php b/resources/lang/en/forms.php
@@ -139,6 +139,7 @@
             'automatic_localization'       => 'Automatically localise your status page to your visitor\'s language?',
             'enable_external_dependencies' => 'Enable Third Party Dependencies (Google Fonts, Trackers, etc...)',
             'show_timezone'                => 'Show the timezone the status page is running in.',
+            'only_disrupted_days'          => 'Only show days containing incidents in the timeline?',
         ],
         'analytics' => [
             'analytics_google'       => 'Google Analytics code',

diff --git a/resources/views/dashboard/settings/app-setup.blade.php b/resources/views/dashboard/settings/app-setup.blade.php
@@ -117,6 +117,17 @@
                                 </div>
                             </div>
                         </div>
+                        <div class="row">
+                            <div class="col-xs-12">
+                                <div class="checkbox">
+                                    <label>
+                                        <input type="hidden" value="0" name="only_disrupted_days">
+                                        <input type="checkbox" value="1" name="only_disrupted_days" {{ $only_disrupted_days ? 'checked' : null }}>
+                                        {{ trans('forms.settings.app-setup.only_disrupted_days') }}
+                                    </label>
+                                </div>
+                            </div>
+                        </div>
                     </fieldset>
 
                     <div class="row">