• 
      

    Allow users to say "Ship It!" to their own review requests by default.

    Review Request #14402 — Created April 23, 2025 and submitted

    Information

    Review Board
    release-7.1.x

    Reviewers

    In RB 7.1 we added a setting to disallow users from saying "Ship It!" to
    their own review requests. By default we had set this to disallow
    because it makes sense that users shouldn't approve their own changes.
    However, its a noticeable change from what we had before and I think
    it'll surprise existing users too much. I myself got confused and wondered
    where the Ship It! button went. This change enables this setting by default.

    • Ran unit tests.
    • Toggled the setting and saw that I couldn't self ship-it when disabled,
      saw that I could when enabled.
    Summary ID
    Allow users to say "Ship It!" to their own review requests by default.
    In RB 7.1 we added a setting to disallow users from saying "Ship It!" to their own review requests. By default we had set this to disallow because it makes sense and is reasonable in practice. However, its a noticeable change from what we had before and I think it'll surprise existing users too much. I myself got confused and wondered where the ship it button went. This change enables this setting by default.
    1d078ef334c2a42da30d2aec8919018d8f35d6af
    chipx86
    1. Ship It!
    2. 
        
    david
    1. Ship It!
    2. 
        
    maubin
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-7.1.x (0a0eeb0)