This changeset gives a better error message when no IRC channel is configured, and adds the IRC widget by default for group hubs.
Should the widget append() be inside the if statement?
I recommend a few tests to assert this if statement works as expected.
Oh I suppose this is why we don't want that append inside the if block above.
LGTM
rebased onto 747043bdcf3529a103881e00a07e91536de9a975
Pull-Request has been merged by abompard
This changeset gives a better error message when no IRC channel is configured, and adds the IRC widget by default for group hubs.