Skip to content

Conversation

@ehhc
Copy link
Contributor

@ehhc ehhc commented Jul 30, 2018

No description provided.

@sosukesuzuki sosukesuzuki added the awaiting review ❇️ Pull request is awaiting a review. label Jul 31, 2018
@ZeroX-DG
Copy link
Member

ZeroX-DG commented Aug 2, 2018

LGTM 👍

@Rokt33r Rokt33r added next release (v0.11.9) awaiting review ❇️ Pull request is awaiting a review. and removed awaiting review ❇️ Pull request is awaiting a review. next release (v0.11.9) labels Aug 9, 2018
@Rokt33r
Copy link
Member

Rokt33r commented Aug 9, 2018

@ehhc Could you tell me what does this PR? I couldn't understand which problem is fixed by this PR.

@ehhc
Copy link
Contributor Author

ehhc commented Aug 9, 2018

If you export a note that contained images as a markdown file, the attachments/images should be exported as well an the links to these files should be valid (instead of :storage/...). If you open the exported note in a markdown preview program you should be able to view the content inclusive the rendered images/files.

@ehhc
Copy link
Contributor Author

ehhc commented Aug 9, 2018

see #1786

@Rokt33r
Copy link
Member

Rokt33r commented Aug 9, 2018

@ehhc I see the point! It looks good to me!

@Rokt33r Rokt33r merged commit 39c9574 into BoostIO:master Aug 9, 2018
@Rokt33r Rokt33r added next release (v0.11.9) and removed awaiting review ❇️ Pull request is awaiting a review. labels Aug 9, 2018
@ehhc ehhc deleted the Attachments_in_markdown_export_#1786 branch August 9, 2018 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants