Merge lp:~jakedahn/django-nova/fb816 into lp:django-nova

Proposed by Jake Dahn
Status: Merged
Approved by: Devin Carlen
Approved revision: 17
Merged at revision: 17
Proposed branch: lp:~jakedahn/django-nova/fb816
Merge into: lp:django-nova
Diff against target: 11 lines (+0/-1)
1 file modified
src/django_nova/templates/django_nova/keypairs/index.html (+0/-1)
To merge this branch: bzr merge lp:~jakedahn/django-nova/fb816
Reviewer Review Type Date Requested Status
Devin Carlen Approve
Review via email:

Description of the change

This branch fixes the error where firefox, and internet explorer were not asking for confirmation when attempting to delete a keypair.

Confirmations should occur after killing the stray console.log.

To post a comment you must log in.
Revision history for this message
Devin Carlen (devcamcar) wrote :

Nice find!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/django_nova/templates/django_nova/keypairs/index.html'
2--- src/django_nova/templates/django_nova/keypairs/index.html 2011-01-17 05:55:41 +0000
3+++ src/django_nova/templates/django_nova/keypairs/index.html 2011-02-27 11:36:09 +0000
4@@ -48,7 +48,6 @@
5 $(function() {
6 $('.form-key-delete').submit(function() {
7 _key_name = $(this).children(':first').val()
8- console.log(_key_name);
9 $('#spn_delete_key_name').text(_key_name);
10 $('#dlg_confirm').dialog('open');
11 return false;


People subscribed via source and target branches

to all changes:
to status/vote changes: