Skip to content
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.

Sign up for GitHub

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

Jump to bottom

md_line Twig filter #37

Closed
wants to merge 1 commit into from
Closed

md_line Twig filter #37

wants to merge 1 commit into from

Conversation

matteotrubini
Copy link
Contributor

@matteotrubini matteotrubini commented Dec 7, 2021 •
edited
Loading

See wintercms/winter#370 - to be merged on v1.1.8 release

@LukeTowers
Copy link
Member

Just include this one and the md_safe ones on the | md filter page instead of creating a new page for it please. Side note, if new pages are created they need to be added to the table of contents in the config folder as well.

@matteotrubini
Copy link
Contributor Author

Done.

PS: maybe makes sense also to add | md_clean to expose Markdown::parseClean() ? See wintercms/storm@2d8fb57

@LukeTowers
Copy link
Member

Sounds good to me

LukeTowers added a commit that referenced this pull request Feb 7, 2022
@LukeTowers
Improve documentation for available filters
f99a0ab 
Replaces #37
@LukeTowers LukeTowers closed this Feb 7, 2022
@LukeTowers
Copy link
Member

Had to merge in f99a0ab, force pushing breaks our merge flow @matteotrubini. Thanks for the contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Status: Completed Type: Maintenance
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@matteotrubini @LukeTowers