#486 Remove some options from user config dialog
Merged by abompard. Opened by ryanlerch.
ryanlerch/fedora-hubs remove-avatar-from-user-config  into  develop

Download 486.patch
  • remove the avatar option in userhub config #470
  • remove hub visibility option from userhub config #484
  • remove user management tabs from userhubs config #483
  • comment out the other tab that said it wasnt implemented #485

Signed-off-by: Ryan Lerch rlerch@redhat.com

On a user hub, there's only the General and IRC tabs left. Do you think the IRC tab is relevant for a user hub, or should it be left for team hubs too?

yeah, i wasnt sure about this one, i'll remove it for now.

that will leave a single tab on the user one, but with the future config changes we discussed last night, i assume the user config will get more than one tab in the future. So i wont bother removing the tab sidebar in that modal for now.

rebased onto 8977948b4077054ac7709d1685632d11360b8584

rebased and edited the commit.

I thought the exact same thing. Thanks, looks good :-)

Pull-Request has been merged by abompard

Metadata