Skip to content
This repository was archived by the owner on Oct 15, 2024. It is now read-only.

Comments

fallback to site title if page title is not set for og:title tag#288

Open
intoeetive wants to merge 1 commit intonystudio107:developfrom
intoeetive:develop
Open

fallback to site title if page title is not set for og:title tag#288
intoeetive wants to merge 1 commit intonystudio107:developfrom
intoeetive:develop

Conversation

@intoeetive
Copy link
Contributor

Fix to avoid trailing | in og:title / twitter:title tags in page title is empty

@benface
Copy link
Contributor

benface commented Oct 25, 2017

Thank you! I was about to submit exactly the same PR haha.

@benface
Copy link
Contributor

benface commented Nov 10, 2017

@khalwat Have you seen this?

@khalwat
Copy link
Collaborator

khalwat commented Nov 10, 2017

Yep, I've seen it. I just need to do some local testing before I can safely merge it as a release.

I've had a few PRs that ended up causing problems, so I try to be conservative about testing them.

@benface
Copy link
Contributor

benface commented Mar 14, 2018

@khalwat Bump! Hehe

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants