-
Notifications
You must be signed in to change notification settings - Fork 92
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
Change asset enqueuing #419
Conversation
joshcanhelp
commented
Mar 27, 2018
•
edited
Loading
edited
- Move asset URLs to constants
- Combine multiple instances of scripts and styles being enqueued in the admin to one place
- Combine multiple instances of Lock URLs to one place
20940aa
to
0e3a49e
Compare
} | ||
|
||
public function shortcode( $atts ) { | ||
wp_enqueue_script( 'jquery' ); | ||
|
||
if ( WP_Auth0_Options::Instance()->get('passwordless_enabled') ) { |
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.
Moved to templates/login-form.php
|
||
wp_enqueue_script( 'jquery' ); | ||
|
||
if ( WP_Auth0_Options::Instance()->get('passwordless_enabled') ) { |
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.
Moved to templates/login-form.php
|
||
if ( trim( $client_id ) !== '' ) { | ||
|
||
if ( WP_Auth0_Options::Instance()->get( 'passwordless_enabled' ) ) { |
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.
Moved to templates/login-form.php
0e3a49e
to
3d9d42c
Compare
@@ -31,12 +31,6 @@ public function show_error() { | |||
|
|||
public function render_import_settings_page() { | |||
|
|||
wp_enqueue_media(); |
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.
Moved to lib/admin/WP_Auth0_Admin.php
3d9d42c
to
ee9c038
Compare
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.
LGTM
89e7416
to
07cc0ab
Compare
Non-breaking change to centralize asset URLs and remove duplication
07cc0ab
to
979bfc6
Compare