----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard-tflink.rhcloud.com/r/17/#review19 -----------------------------------------------------------
Looks great overall. I had a few more comments but they're minor and I don't think that their fixes will require another review - feel free to merge into develop after you're done with the comments unless you have concerns about them. blockerbugs/controllers/users.py <http://reviewboard-tflink.rhcloud.com/r/17/#comment31> I don't think this is enough to stop these changes but we should probably start using the flash messages in other places than the bz association page. If you've logged in and out before doing the association, the flash messages pile up and you end up with multiple login successful messages on the bz account association page blockerbugs/templates/fas_bugzilla.html <http://reviewboard-tflink.rhcloud.com/r/17/#comment32> I'm being a little nit-picky here but the wording of this title is a bit akward. What about "Link your FAS and Red Hat Bugzilla Accounts"? blockerbugs/templates/fas_bugzilla.html <http://reviewboard-tflink.rhcloud.com/r/17/#comment30> can you either remove the fieldset around this login or add it to the FAS login page to make them consistent? blockerbugs/templates/fas_bugzilla.html <http://reviewboard-tflink.rhcloud.com/r/17/#comment33> Can you stick a bit of text at the end of this which says: "Your Red Hat bugzilla password is used only to verify that you control the account which will be linked to your FAS account. Your password will <b>NOT</b> be stored anywhere in this application." - Tim Flink On March 14, 2013, 11:24 a.m., Martin Krizek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard-tflink.rhcloud.com/r/17/ > ----------------------------------------------------------- > > (Updated March 14, 2013, 11:24 a.m.) > > > Review request for blockerbugs. > > > Bugs: 347 > https://fedorahosted.org/fedora-qa/ticket/347 > > > Repository: blockerbugs > > > Description > ------- > > This is a patch for tickets 347, 348, 349 and 350. > > > Diffs > ----- > > testing/test_bugchange.py 65f291122b4ae687872170517d196a621d548152 > blockerbugs/util/bz_interface.py a8958be674e839d6c99694e33440a26331a8b041 > blockerbugs/templates/propose_bug.html > c97d72303dff2f49e4bb8bc5ff5fc2f4c764ce8b > blockerbugs/templates/login.html 20674fe93eb96c6e1e9b12b5988cad67dc1a5181 > blockerbugs/templates/layout.html 6d0ae9a7bc913aa408985028db4494d51379da9c > blockerbugs/templates/fas_bugzilla.html PRE-CREATION > blockerbugs/templates/base_nav.html > ce7a7bb478ad57a65d646fa7326a210c6a4eb131 > blockerbugs/models/userinfo.py PRE-CREATION > blockerbugs/controllers/users.py 30b5fb5c8ef2196979a2d22c4380f302a5ce951a > blockerbugs/controllers/main.py ce26c641b9db05f11510f16177b017a56cf757c2 > blockerbugs/controllers/forms.py b213c055d9297b59c6cf5d83358dcd14e265e345 > alembic/versions/1d12b74d12bd_add_userinfo_table.py PRE-CREATION > > Diff: http://reviewboard-tflink.rhcloud.com/r/17/diff/ > > > Testing > ------- > > > Thanks, > > Martin Krizek > >
_______________________________________________ qa-devel mailing list [email protected] https://admin.fedoraproject.org/mailman/listinfo/qa-devel
