Update the MR review policy for design review


#1

I suggest an update to the MR review policy (currently there’s no real policy, just accountabilities mentioned at the bottom of https://git.snowdrift.coop/sd/governance/blob/master/Website-Circle.md).

In the case of significant design overhauls (like swapping out a page, adding a new page that wasn’t there, or major UI modifications), such work should not be merged and deployed until reviewed by the design roles (so, @msiep and @mray ). Exceptions for anything urgent, of course. And maybe specify some time frame (48 hours?) after which we may move forward even without review (because we don’t want to let MRs get stale or stalled).


#2

That sounds reasonable but I think it should be @mray or @msiep reviewing the MR. Given availability, I don’t think waiting for both of them is practical (though we certainly can if there’s no rush to merge).