Skip to content
This repository has been archived by the owner on May 27, 2020. It is now read-only.

Fixed preview of the blog_post content draft #42

Merged

Conversation

clash82
Copy link
Contributor

@clash82 clash82 commented Sep 14, 2016

Should be included with 91a2c13.

Fixes issue with blog_post preview after implementing QueryType.

@clash82
Copy link
Contributor Author

clash82 commented Sep 14, 2016

ping @SylvainGuittard @damianz5

@clash82 clash82 changed the title Fixed blog_post content preview Fixed preview of the blog_post content draft Sep 14, 2016
@clash82 clash82 force-pushed the fix-blog-post-content-preview branch from 46e8cbc to 9beade5 Compare September 14, 2016 14:36
@SylvainGuittard
Copy link
Contributor

+1

1 similar comment
@damianz5
Copy link
Contributor

+1

@SylvainGuittard SylvainGuittard merged commit b0c6f2a into ezsystems:master Sep 15, 2016
@clash82 clash82 deleted the fix-blog-post-content-preview branch September 15, 2016 13:12
@andrerom
Copy link
Contributor

andrerom commented Oct 12, 2016

(side: in retrospective this should have been applied on 1.5 branch and up)

@clash82
Copy link
Contributor Author

clash82 commented Oct 12, 2016

(side: in retrospective this should have been applied on 1.5 branch and up)

Yes, that's true :(
@andrerom should I cherry-pick this commit on 1.5 branch or is there a more proper way?

@andrerom
Copy link
Contributor

andrerom commented Oct 12, 2016

not a big must as 1.6.0 is soon here, but you can do that, just in case we do a 1.5.2 update to sales demo and/or platform.sh, but make sure to merge 1.5 into master so they stay in sync.

@clash82
Copy link
Contributor Author

clash82 commented Oct 13, 2016

@andrerom I was wrong. This commit should belong to master only as it fixes issue with blog_post preview but after implementing QueryTypes (added in 1.6).

This one here 91a2c13 fixes same issue but before QueryTypes. So partially it's related (fixes same template) but should stay on master at this moment.

Description was misleading, sorry for that.

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

Successfully merging this pull request may close these issues.

4 participants