need for mergeHtmlAndAttachment in markdown resolved() method? about woofmark

[jywarren]

Could we insert a catch here:

woofmark/src/markdown/linkOrImageOrAttachment.js

Line 119 in b58efb7

links.forEach(renderLink);

and defer to a custom mergeHtmlAndAttachment or corresponding mergeMarkdownAndAttachment at that point instead of going to renderLink?

@bevacqua what do you think about this way forward? Thanks!

[bevacqua]

It sounds like you want a different function that lets you add custom handling of links, that could be used internally to manage attachments, and exposed so that people can add their own kinds of special "attachment" links

[jywarren]

that sounds reasonable -- then it could be checked for and used in both "markdown" and "html" code -- how about `formatAttachment()`? -- thanks!!

…

On Sun, May 6, 2018 at 9:48 AM, Nicolás Bevacqua ***@***.***> wrote: It sounds like you want a different function that lets you add custom handling of links, that could be used internally to manage attachments, and exposed so that people can add their own kinds of special "attachment" links — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#67 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AABfJ-3RvuW7a1zO1WK-09spwKGjCLj5ks5tvv81gaJpZM4TvkPO> .