Skip to content
This repository has been archived by the owner on Dec 17, 2018. It is now read-only.

Modified get_meta_box_url() to return a protocol agnostic uri #639

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Blindmikey
Copy link

HTTPS environments fail to load the asset on certain browsers due to http:// being a part of the uri. By being protocol agnostic, we play well with all environments.

HTTPS environments fail to load the asset on certain browsers due to http:// being a part of the uri. By being protocol agnostic, we play well with all environments.
@jtsternberg
Copy link
Contributor

I like this. However, can you please keep with the contribution guidelines, namely submit PR's against the trunk branch, and follow WP (and CMB) coding standards? Thank you kindly.

@Blindmikey
Copy link
Author

Awesome, I'd be happy to - will just have to read up on those guidelines. Will have more time over the weekend.

Yay for my first PR!

@jtsternberg
Copy link
Contributor

thanks @Blindmikey! Now you're unstoppable. ;)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants