You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@whimsical.apache.org by Sebastian Bazley <se...@apache.org> on 2016/03/14 11:46:00 UTC
[whimsy.git] [1/1] Commit 8e9ebbc: Don't let title hide data
Commit 8e9ebbc49bd0650c9d2d389982d0e207ed27026f:
Don't let title hide data
Branch: refs/heads/master
Author: Sebb <se...@apache.org>
Committer: Sebb <se...@apache.org>
Pusher: sebb <se...@apache.org>
------------------------------------------------------------
www/status/monitors/public_json.rb | +++++
------------------------------------------------------------
5 changes: 5 additions, 0 deletions.
------------------------------------------------------------
diff --git a/www/status/monitors/public_json.rb b/www/status/monitors/public_json.rb
index 9c4e8de..9fcbd5d 100644
--- a/www/status/monitors/public_json.rb
+++ b/www/status/monitors/public_json.rb
@@ -63,6 +63,11 @@ def Monitor.public_json(previous_status)
unless contents.empty?
status[name].merge! level: 'danger', data: contents.split("\n")
end
+ # monitor.rb ignores data if title is set
+ # TODO: is this a bug in monitor.rb ?
+ if status[name][:data]
+ status[name].delete_if { |k, v| k.eql? :title}
+ end
rescue Exception => e
status[name] = {
level: 'danger',