Code review comment for lp:~dylanmccall/harvest/harvest-dylan-m

Revision history for this message
Daniel Holbach (dholbach) wrote :

(minor) Maybe FilterSystem.update_http() could be called FilterSystem.update_from_http() to make clear what is updated based on which data?

Can FilterSystem and FilterContainer be merged? Are they used separately and in different ways? I must admit FilterSystem is not quite clear to me yet. Still digging through it. :)

« Back to merge proposal