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

Fix #1879 -- use user’s HTML for custom teaser text #1880

merged 1 commit into from Jul 10, 2015

Conversation

@Kwpolska
Copy link
Member

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>
@Kwpolska Kwpolska added this to the v7.6.1 milestone Jul 9, 2015
@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

Kwpolska commented Jul 10, 2015

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
3 checks passed
3 checks passed
continuous-integration/appveyor AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@Kwpolska Kwpolska deleted the fix-1879 branch Jul 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.