-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Add suppport for image links to be previewed #504
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dmeza looks good, just one comment, we should remove the "Get Public Link" option for these previews (if it's enabled in the config) since the link is coming directly from the post and it doesn't make much sense.
@miguelespinoza please see the comment about "Get Public Link". |
@esethna I've added the fix for that 😄 |
28e6ee6
to
358474f
Compare
358474f
to
d56bb85
Compare
@lfbrock please help us test these PR. It's rebased to the latest from master. |
|
Spinmint test server created at: http:https://i-0c13b036dec81909f.spinmint.com Test Account 1: Email: Test Account 2: Email: Instance ID: i-0c13b036dec81909f |
Changes have been made, Eric is away so dismissing review since he's not here to approve
Spinmint test server destroyed |
* Add suppport for image links to be previewed * Do not display Public Link for inline link images * updated snapshots
* Add suppport for image links to be previewed * Do not display Public Link for inline link images * updated snapshots
* Add suppport for image links to be previewed * Do not display Public Link for inline link images * updated snapshots
* Add suppport for image links to be previewed * Do not display Public Link for inline link images * updated snapshots
* Add suppport for image links to be previewed * Do not display Public Link for inline link images * updated snapshots
* fix combined posts which ignores custom and other post types * removed console log and added assertions of posts Signed-off-by: Saturnino Abril <[email protected]>
* fix combined posts which ignores custom and other post types * removed console log and added assertions of posts Signed-off-by: Saturnino Abril <[email protected]>
Summary
Preview link images just like you do with images uploaded from MM.
Checklist
Screenshots