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

Fix no empty twitter tags #120

Merged
merged 3 commits into from
Aug 19, 2013
Merged

Conversation

jrfnl
Copy link
Contributor

@jrfnl jrfnl commented Jul 20, 2013

Checked all twitter and opengraph methods for:

  • proper html, i.e. "..." not '...' for attributes
  • avoiding sending empty tags - especially with all the filters, you never know whether they are used properly
  • proper use of esc_attr()/esc_url()

Seen various questions on the forum about this already, so this should shut them up ;-)

* proper html, i.e. "..." not '...' for attributes
* avoiding sending empty tags - especially with all the filters, you never know whether they are used properly
* proper use of esc_attr()/esc_url()
jrfnl added a commit that referenced this pull request Aug 19, 2013
@jrfnl jrfnl merged commit 03cd841 into Yoast:master Aug 19, 2013
@jrfnl jrfnl deleted the Fix-no-empty-twitter-tags branch August 19, 2013 03:19
hansjovis pushed a commit that referenced this pull request Nov 21, 2022
Added doc regarding new filter for Organization URL in Local SEO 13.9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant