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 #1879 -- use user’s HTML for custom teaser text #1880

Merged
merged 1 commit into from Jul 10, 2015
Merged

Conversation

Kwpolska
Copy link
Member

@Kwpolska Kwpolska commented Jul 9, 2015

This is #1879.

Are there any places where this could break? This could bite, we might need some other solution (checking if INDEX_READ_MORE_LINK makes sense?)

cc @notfoss, @ralsina

Signed-off-by: Chris Warrick kwpolska@gmail.com

Signed-off-by: Chris Warrick <kwpolska@gmail.com>
@notfoss
Copy link
Contributor

notfoss commented Jul 10, 2015

checking if INDEX_READ_MORE_LINK makes sense?

Can you elaborate on that? Do you want to do away with this option?

@Kwpolska
Copy link
Member Author

No, not at all. The “check” would be to make sure the read more… link can render something sensible (it’s not empty). That would be a corner case; we can probably just ignore it.

Merging.

Kwpolska added a commit that referenced this pull request Jul 10, 2015
Fix #1879 -- use user’s HTML for custom teaser text
@Kwpolska Kwpolska merged commit 1c1a6f4 into master Jul 10, 2015
@Kwpolska Kwpolska deleted the fix-1879 branch July 10, 2015 07:33
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

2 participants