-
Notifications
You must be signed in to change notification settings - Fork 116
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update process tweaks #450
Conversation
Conflicts: includes/admin.php
Conflicts: includes/admin.php
Add license-remove functionality Tweak UI Add loading spinner for ajax operations
Conflicts: readme.md readme.txt
Conflicts: ui/admin.css
…connect to premium
…ckage urls correctly
- CSS is for making the video container responsive
@shadyvb Looks like there are some conflicts to be resolved before merging. |
Conflicts: includes/admin.php ui/admin.css
@fjarrett Voila! |
load : function() { | ||
var template = $('.theme-wrap'); | ||
var overlay = $('.theme-overlay'); | ||
var ext_obj = stream_extensions.extensions; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shadyvb ui/extensions.js: line 14, col 28, 'stream_extensions' is not defined.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Or maybe @c3mdigital?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fjarrett It's defined in the call to wp_localize_script()
in https://github.com/x-team/wp-stream/pull/450/files#diff-7eec5692b9fd9edd9bf006170f5ce1edR481
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@c3mdigital Travis is flagging it: https://travis-ci.org/x-team/wp-stream/jobs/23701716#L240
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This has been merged into #451 and will be handled there. Closing. |
No description provided.