Description of problem: As a group owner, I tried to add new members to an existing beaker group but failed. Version-Release number of selected component (if applicable): 22.1 How reproducible: always Steps to Reproduce: 1. go to beaker group member page - e.g. https://beaker.engineering.redhat.com/groups/kernel-general-qe 2. input user krb id in "Username" field and select the right one from filtered list 3. click "+ Add" Actual results: page reloaded but group member list remained the same, no warning/error message either Expected results: new member got added into group member list Additional info:
(In reply to Linqing Lu from comment #0) > Description of problem: > As a group owner, I tried to add new members to an existing beaker group but > failed. > > Version-Release number of selected component (if applicable): > 22.1 > > How reproducible: > always > > Steps to Reproduce: > 1. go to beaker group member page - e.g. > https://beaker.engineering.redhat.com/groups/kernel-general-qe > 2. input user krb id in "Username" field and select the right one from > filtered list > 3. click "+ Add" > > Actual results: > page reloaded but group member list remained the same, no warning/error > message either > > Expected results: > new member got added into group member list > > Additional info: Hi Linqing Lu, Thank you for reporting but I cannot reproduce it. There is no user with user id krb. Could you please provide more information? Thanks
Besides, the browser was Chrome v44.
(In reply to Linqing Lu from comment #3) > Besides, the browser was Chrome v44. Okay, we donnot actually support Chrome very well as we think most of the users are using firefox. I have tried it on firefox and it works so please use it as a workaround.
On Gerrit: http://gerrit.beaker-project.org/#/c/4699/
So the problem was we were using some ES6 features not supported by Chrome 44. We can fix it in this case easily but I think we are relying on ES6 support in other parts of Beaker's web UI, and will continue to do so more and more since all the supported releases of the major browsers support ES6 now. Bear in mind that Chrome 44 is several releases out of date and is unsupported by Google, so you should consider upgrading.
(In reply to Dan Callaghan from comment #6) > So the problem was we were using some ES6 features not supported by Chrome > 44. We can fix it in this case easily but I think we are relying on ES6 > support in other parts of Beaker's web UI, and will continue to do so more > and more since all the supported releases of the major browsers support ES6 > now. > > Bear in mind that Chrome 44 is several releases out of date and is > unsupported by Google, so you should consider upgrading. Yes, we are using Array.find() from ES6 which is not supported by Chrome 44. In my above patch, I change it to underscore.find() that fixes the problem.
Beaker 22.2 has been released.