-
-
Notifications
You must be signed in to change notification settings - Fork 63
Prep Plugin for release on WordPress.org #23
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
Conversation
Escape everything that should be escaped. Add nonce checks where needed. Sanitize all inputs. Apply Code style changes across the codebase. Correct many deprecation notices. Optimize load order of many filters.
# Conflicts: # inc/class-whitelabel.php
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.
correct
|
Amazing work! Gonna give it all a good test. 🙏 |
|
@marcusquinn I was going to just merge it as-is and correct any problems with additional PRs. But if you want to give it a good review and test first that'd be great. I know there are issues. So far checkout doesn't work because of Vue problems. It probably related to escaping something which shouldn't be escaped. It could also be moving js to the footer and the related timing problems. |
Escape everything that should be escaped.
Add nonce checks where needed.
Sanitize all inputs.
Apply Code style changes across the codebase.
Correct many deprecation notices.
Optimize load order of many filters.