Skip to content

Loading…

Fixes Issue #18 #19

Merged
merged 3 commits into from

2 participants

@kracekumar

Mouse click inside modal window doesn't hide the playlist form.

@kracekumar

While creating new ajax playlist, video are added automatically to playlist, but session commit statement was missing.

@jace jace merged commit 2d00251 into master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Showing with 2 additions and 3 deletions.
  1. +1 −3 hgtv/templates/video.html
  2. +1 −0 hgtv/views/channel.py
View
4 hgtv/templates/video.html
@@ -278,9 +278,7 @@
toastr.error("something went wrong try later")
}
});
- } else {
- $(modalWindow).modal('hide');
- }
+ }
}
});
}
View
1 hgtv/views/channel.py
@@ -80,6 +80,7 @@ def playlist_new_modal(channel, video):
message_type = 'info'
action = 'noop'
html_to_return = render_template('new-playlist-tag.html', playlist=playlist, channel=channel, video=video)
+ db.session.commit()
return jsonify({'html': html_to_return, 'message_type': message_type, 'action': action,
'message': message})
if form.errors:
Something went wrong with that request. Please try again.