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

WordPress plugin fixes #103

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

WordPress plugin fixes #103

wants to merge 7 commits into from

Conversation

da2x
Copy link
Contributor

@da2x da2x commented Feb 15, 2017

WordPress Plugin Directory requires all functions and options to
use a unique prefix to avoid collision with core or other plugins.
No upgrade code, so notify users in update message.

Removed generic 'Tipsy' string in favor of the blog's own name.
Could use author name, but would require more logic for pages
without a given author or multi-author pages.

Use esc_html to escape user input.

@tbranyen
Copy link
Member

Looks good to me!

WordPress Plugin Directory requires all functions and options to
use a unique prefix to avoid collision with core or other plugins.
No upgrade code, so notify users in update message.

Removed generic 'Tipsy' string in favor of the blog's own name.
Could use author name, but would require more logic for pages
without a given author or multi-author pages.

Use `esc_html` to escape user input.
* Promote browser extension
* Reassure it will work with all themes
* Correct URI
* More attractive tags
Main file must have the same name as the parent directory. This becomes
the plugin's URI slug when uploaded to the WordPress Plugin Directory.
@da2x da2x changed the title Unique prefix for options and functions WordPress plugin fixes Feb 18, 2017
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.

None yet

2 participants