Skip to content

Loading…

Top 5 active users broken #233

Closed
corelanc0d3r opened this Issue · 6 comments

2 participants

@corelanc0d3r

After enabling gravatar for one admin, and disabling it again, the top 5 active user widget on the dashboard contains a combination of names and /results? entries

@mephux
Snorby member

I'm looking into this tomorrow - i'll keep you posted.

@mephux mephux closed this
@corelanc0d3r

closed = fixed, or just closed ?

@mephux
Snorby member

yes, this should be fixed in the latest version. Are you still having this issue?

@mephux mephux reopened this
@corelanc0d3r

I'll test

@corelanc0d3r

ran git pull, restarted webserver... issue is still there

git pull log :

remote: Counting objects: 53, done.
remote: Compressing objects: 100% (19/19), done.
remote: Total 36 (delta 26), reused 27 (delta 17)
Unpacking objects: 100% (36/36), done.
From https://github.com/Snorby/snorby
   8b97843..52a5b98  master     -> origin/master
Updating 8b97843..52a5b98
Fast-forward
 app/controllers/sensors_controller.rb   |    5 +
 app/views/notes/_add_note.html.erb      |   23 +++---
 app/views/notes/_note.html.erb          |   29 +++---
 app/views/sensors/_sensors.html.erb     |   22 +++++
 app/views/sensors/index.js.erb          |    2 +
 app/views/users/passwords/edit.html.erb |    6 +-
 public/javascripts/snorby.js            |   18 +++--
 public/stylesheets/snorby.css           |  148 ++++++++++++++++---------------
 8 files changed, 147 insertions(+), 106 deletions(-)
 create mode 100644 app/views/sensors/_sensors.html.erb
 create mode 100644 app/views/sensors/index.js.erb
@corelanc0d3r

is someone still looking at this ?

@mephux mephux closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.