Is this for testing?

An == comparaison seems more conventional here (both will work).

This is so common we should probably add it to the env filters for all widgets (in hubs/widgets/base.py. However, the others expect a date object (they are defined as lambda d: arrow.get(d).humanize()) and this one expects an Arrow object. This should be uniformized if we register this filter globally.

thanks for the review @abompard!

Have removed the invalidation stuff -- this was indeed for testing purposes -- sorry!

made that comparision == rather than is

Also, i agree with abstracting the humanize stuff out -- do you mind if i do it in a PR after this one, just so i'm not touching a bunch of other widgets in this PR?

Could you remove the last horizontal separator? I see you removed it in the CSS but it does not work because the last child of the parent element is actually the <style> tag. Moving that tag to the top works :-)

Also, do we want this widget by default for team hubs? I guess it would make sense since we have the IRC widget by default. Then a message should be displayed if no mailing-list has been configured in the settings.

How about also adding a "Start a new thread" button at the bottom, which would externally link to https://lists.fedoraproject.org/archives/list/<list-address>/message/new ? Like the "Create a new meeting" button for the meetings widget.

Added the message to show if no mailing list has been set.

Also added the new thread button at the bottom. Left the seperator in there (removed that CSS all together) as it looked good with the line at the bottom when the button was added.

Commit 039f54f8 fixes this pull-request