• 
      

    Update the My Account page documentation and add new settings.

    Review Request #15064 — Created May 17, 2026 and submitted

    Information

    Review Board
    release-8.x

    Reviewers

    The My Account page docs were pretty out-of-date, lacking changes made
    since Review Board 7. It was also fairly inconsistent on how it
    presented available options and descriptions.

    This reworks sections of the page to provide more guidance and less
    dense text. It also adds the missing settings, including:

    • The new Ship It! confirmation and spell check settings.

    • The Appearance page, added in Review Board 7.

    Anchors have been added throughout the page to facilitate linking to
    sections from other pages and release notes.

    Built the docs. Checked for bad links and build issues.

    Summary ID
    Update the My Account page documentation and add new settings.
    The My Account page docs were pretty out-of-date, lacking changes made since Review Board 7. It was also fairly inconsistent on how it presented available options and descriptions. This reworks sections of the page to provide more guidance and less dense text. It also adds the missing settings, including: * The new Ship It! confirmation and spell check settings. * The Appearance page, added in Review Board 7. Anchors have been added throughout the page to facilitate linking to sections from other pages and release notes.
    c2409b5b81272b1ce3be3c29b1cb5b34b85ac4fe
    Description From Last Updated

    Existing field labels in this file don't have trailing colons. Your newly added ones are a mix--some do, some don't.

    david david

    Don't these different formatl (compact dl vs multi-paragraph list items) render differently? We should probably add a blank line to …

    david david

    The 2nd sentence sounds a bit weird, maybe use "different" instead of "other"? Also theres no trailing period.

    maubin maubin

    This should probably refer to the fact that this entire page was added in 7.0.

    david david

    I know that this is what the label says, but looking here, I wonder if we should change that to …

    david david

    This page is now much more than just changing password (also includes API tokens, OAuth tokens)

    david david

    This should probably have a _my-account-change-password anchor

    david david

    You're mixing bulleted lists and numbered lists here.

    david david
    maubin
    1. 
        
    2. docs/manual/users/settings.rst (Diff revision 1)
       
       
       
      Show all issues

      The 2nd sentence sounds a bit weird, maybe use "different" instead of "other"? Also theres no trailing period.

      1. I think I was trying to decide how I wanted this sentence to go and got distracted.

    3. 
        
    david
    1. 
        
    2. docs/manual/users/settings.rst (Diff revision 1)
       
       
       
       
       
       
       
       
       
       
      Show all issues

      Existing field labels in this file don't have trailing colons. Your newly added ones are a mix--some do, some don't.

    3. docs/manual/users/settings.rst (Diff revision 1)
       
       

      I believe it should be fine with the intersphinx setup, but just wanted to call out that the old name here is still in the 3.0 release notes.

      1. Should be fine, but doesn't hurt to keep the old name around.

    4. docs/manual/users/settings.rst (Diff revision 1)
       
       
       
       
       
       
       
       
       
       
       
       
       
      Show all issues

      Don't these different formatl (compact dl vs multi-paragraph list items) render differently? We should probably add a blank line to the existing ones so they all render as paragrphs.

      1. Definitely incorrect before. Fixing them.

    5. docs/manual/users/settings.rst (Diff revision 1)
       
       
      Show all issues

      This should probably refer to the fact that this entire page was added in 7.0.

    6. docs/manual/users/settings.rst (Diff revision 1)
       
       
       
      Show all issues

      I know that this is what the label says, but looking here, I wonder if we should change that to be "Match system theme"?

      1. Probably. It started out being Light Mode by default, which is why it was named that way.

        I took a quick stab at it and think it's too late to make a change here for this release. It's not a list of hard-coded items but rather something based on the registry, where it grabs the default, populates the first entry, and then adds remaining entries. This is then all based on siteconfig state and some other logic. Doing it right is more involved than changing a string, and we'd need to figure out whether we'd want to keep the previous UI if the default theme is different. Today that's unlikely, but yeah, lots of work with considerations to make, and best done in a future release.

    7. docs/manual/users/settings.rst (Diff revision 1)
       
       
       
       
      Show all issues

      This page is now much more than just changing password (also includes API tokens, OAuth tokens)

    8. docs/manual/users/settings.rst (Diff revision 1)
       
       
       
      Show all issues

      This should probably have a _my-account-change-password anchor

    9. docs/manual/users/settings.rst (Diff revision 1)
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
       
      Show all issues

      You're mixing bulleted lists and numbered lists here.

      1. Correct. Some are multi-step processes (creating a token, removing a token). Editing is not. Those are different options that can be performed.

    10. 
        
    chipx86
    david
    1. Ship It!
    2. 
        
    chipx86
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-8.x (d3f80b2)