Skip to content
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

some changes #88

Open
wants to merge 60 commits into
base: master
Choose a base branch
from
Open

some changes #88

wants to merge 60 commits into from

Conversation

ttodua
Copy link

@ttodua ttodua commented Feb 10, 2018

plugin had some coding mistakes, that causes the whole plugin to brake on Active24.com hosting servers. Fixed.

didn't realize that I was giving information about my sites to private
data collection until I reviewed the codebase. not cool.
plugin updating is done by wordpress itself, addons don't need a tab in
the plugin itself, and license management/verification is a waste of my
bandwidth
move license into separate file
add a prettier donate link
remove the changelog - pretty outdated
also fix some local variable names
add in a short explanation as to why it's renamed
rewrite a bunch of the readme
it was removed from the distribution, and I don't really like sass much
anyway
not needed because we don't have an addons/upsell tab
notslang and others added 30 commits June 22, 2014 19:36
used in media-files addon
Bad link referring to old package name.
Previously, this plugin only worked in the /plugins directory. This change makes the .js file location more agnostic to its residence under /plugins or /mu-plugins.
(it screws up syntax highlighting & doesn't need to be in the middle)
Allow plugin to reside in mu-plugins
Saving some memory for echo.
Updating youtube link to the most recent walkthrough. the previous link was the old video with message superimposed "This video has been updated..."
Prevent notices in PHP 7.1 that break admin ui
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants