Rearrange the pages and settings in "My Account".

Review Request #9046 - Created June 29, 2017 and submitted

David Trowbridge
Review Board
release-3.0.x
d8fbb3b...
reviewboard

The various knobs and gewgaws in the account settings had grown organically,
and were kind of a big mess. We had things spread around too many pages and in
somewhat random order.

This change rearranges things so that the user profile information comes first
(name, e-mail address, avatar, etc). Next is settings, which includes general
settings and notifications. Groups is moved to be third on the list (instead of
first), and finally the authentication page has the password and API tokens
together.

  • Ran unit tests.
  • Verified that loading and saving all of the various settings worked correctly.
Loading file attachments...

  • 0
  • 0
  • 1
  • 0
  • 1
Description From Last Updated
David Trowbridge
David Trowbridge
Barret Rennie
  1. This is a great cleanup!

  2. reviewboard/accounts/tests.py (Diff revision 3)
     
     

    Do you want to change this to look it up in the registry, e.g.

    builtin_pages = {
        page.page_id
        for page in AccountPage.registry.get_defaults()
    }
    

    Hardcoding the set of default pages in two places seems less than ideal.

  3. 
      
David Trowbridge
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-3.0.x (4072b5b)
Loading...