-
Notifications
You must be signed in to change notification settings - Fork 72
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
Remove footnotes from og:description
#529
base: master
Are you sure you want to change the base?
Conversation
This change remove any footnotes from the og:description header.
I'm sure there is a better way to filter the pandoc ast, but that fixes my issue. |
og:description
og:description
og:description
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.
@TristanCacqueray I think it is better to fix it at the source itself, so whoever calls this function will benefit:
What do you think?
Sure, thanks, that's much better! |
@TristanCacqueray Have you tested it to work? Because it is not working for me: |
I remember testing the change to heist-extra, but looking at the code it does not seem to make sense (the catchall is already removing unmatched inlines). First of all, the function comes from |
@srid sorry about that, now using |
@TristanCacqueray It would be good to merge this, after that issue above (#529 (comment)) is resolved. |
@srid I agree, unfortunately I don't have much time to work on this at the moment. |
This change remove any footnotes from the og:description header.