#1808 Adding multiple users to group doesn't work

v1.0.0
closed
Kyle Adams
sf-4 (350)
General
nobody
2015-08-20
2011-03-24
Dave Brondsema
No

If I try to add multiple users to a group at once, only the last one adds. Also, if any one of them are not found, they all fail with "User [u'user04', u'user3535'] not found"

Related

Tickets: #3331

Discussion

  • Dave Brondsema

    Dave Brondsema - 2011-04-07
    • size: --> 4
     

  • Rick Copeland - 2011-05-03
    • status: open --> in-progress
    • assigned_to: Rick Copéland
     

  • Rick Copeland - 2011-05-03
    • status: in-progress --> code-review
    • assigned_to: Rick Copéland --> Dave Brondsema
     

  • Rick Copeland - 2011-05-03

    rc/1808

    I was able to add multiple users to a group without errors. I went ahead and updated the bootstrap code so we have test-user-[0-9]. There also seems to be a problem with flash messages that are unrelated to this ticket.

    To QA: add multiple users to a group (use test-user-0 and test-user-1 or some such). They should be added successfully. Now try adding several more where one has an invalid username. They should not be added, and there should be a flash message indicating the first invalid username. (The flash message is display:none'd due to the flash bug above so you'll have to view source).

    (forge)(rc/1808)
: rick446@legolas-vm: ~/src/forge/Allura; git qa
Todays unlucky sucker is: Kyle
     

  • Rick Copeland - 2011-05-03

    force-pushed with a fix to the flash messages

     

  • Rick Copeland - 2011-05-03

    Changing owner because Kyle != Dave

     

  • Rick Copeland - 2011-05-03
    • assigned_to: Dave Brondsema --> Kyle Adams
     

  • Kyle Adams - 2011-05-09

    Freshly built sandboxes are 500'ing out. The odd thing is that I'm not getting the melty anvil (nginx) or an Allura error page. It's just a plain vanilla 500 page. Maybe the melty anvil thing is sfpy only?

    https://sf-kadams-6035.sb.sf.net/p/

     

  • Kyle Adams - 2011-05-09
    • status: code-review --> open
    • assigned_to: Kyle Adams --> Rick Copéland
     

  • Dave Brondsema

    Dave Brondsema - 2011-05-09
    • milestone: may-05 --> may-12
     

  • Rick Copeland - 2011-05-10
    • status: open --> in-progress
     

  • Rick Copeland - 2011-05-11
    • status: in-progress --> code-review
    • assigned_to: Rick Copéland --> Kyle Adams
     

  • Rick Copeland - 2011-05-11

    After finally getting back to this I discovered that the problem is that allura needed rebasing to dev due to some theme changes that were out of sync (on this branch) with sftheme:dev

     

  • Kyle Adams - 2011-05-13

    Looks good.

     

  • Kyle Adams - 2011-05-13
    • status: code-review --> closed
     

Log in to post a comment.