Skip to content
This repository has been archived by the owner on Dec 8, 2019. It is now read-only.

Commit

Permalink
Show connections without subscriptions (fixes #2 and #6)
Browse files Browse the repository at this point in the history
  • Loading branch information
@joakimbeng
joakimbeng committed Sep 16, 2016
1 parent bed7a1f commit ae793d7
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions public/js/app.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -168,7 +168,7 @@ var app = function app() {
content += '<td>' + val.in_msgs + ' / ' + val.out_msgs + '</td>';
content += '<td>' + val.in_bytes + ' / ' + val.out_bytes + '</td>';
content += '<td>' + val.subscriptions + '</td>';
content += '<td>' + val.subscriptions_list.join(', ') + '</td>';
content += '<td>' + (val.subscriptions_list ? val.subscriptions_list.join(', ') : '') + '</td>';

This comment has been minimized.

Sign in to view

Copy link
@cjbhaines

cjbhaines May 25, 2018

Hi @devfacet ,

I'm still seeing this throw using the docker image. Are you planning to release a new image with this fix in it?

https://hub.docker.com/r/urashidmalik/natsboard/

Thanks!
Chris

This comment has been minimized.

Sign in to view

Copy link
@devfacet

devfacet May 26, 2018

Owner

Hi, I don't know who maintains that Docker image. The "official" Dockerfile is https://github.com/devfacet/natsboard/blob/master/Dockerfile Feel free to use that Dockerfile to create an image on https://hub.docker.com/

This comment has been minimized.

Sign in to view

Copy link
@cjbhaines

cjbhaines May 28, 2018

Apologies, I made a bad assumption and should have checked the username on Docker Hub. Thanks for the info!

content += '</tr>';
}
Expand Down Expand Up @@ -409,4 +409,4 @@ var myApp = app();
// Run app
$(document).ready(function() {
myApp.routeHandler(location.pathname);
});
});

0 comments on commit ae793d7

Please sign in to comment.