Review: Needs Fixing Hi Bhavesh,
thanks for your merge proposal! These are useful additions. My major concern is the JS-based confirmation: If someone doesn't have JS activated, the removal will happen immediately. Given that the members are a list's most valued assets that's just a little too risky IMO. How about using a separate confirmation page for that? The text area values could be stored in another form on that page, along with a hidden "confirm" field, so the corresponding view function knows that the user really intends to remove the given members (or all of them). Thanks! Florian -- https://code.launchpad.net/~bhavesh-goyal093/postorius/feature-added-mass-removal/+merge/252595 Your team Mailman Coders is subscribed to branch lp:postorius. _______________________________________________ Mailman-coders mailing list [email protected] https://mail.python.org/mailman/listinfo/mailman-coders
