#413 Some improvements on the basic IRC widget
Merged by abompard. Opened by abompard.
abompard/fedora-hubs feature/irc  into  develop

Download 413.patch

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

Metadata