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

Add and implement markdownify #75

Merged
merged 4 commits into from
Mar 28, 2024
Merged

Add and implement markdownify #75

merged 4 commits into from
Mar 28, 2024

Conversation

joeriddles
Copy link
Contributor

@joeriddles joeriddles commented Mar 23, 2024 •
edited
Loading

Pull Request

Description:

  • Add and implement markdownify to render markdown for events and tech groups
    • markdownify uses bleach to sanitize rendered content

Related Issues:

Checklist:

  • All tests pass.
  • Code follows the project's coding standards.
  • Documentation has been updated.

Screenshots

Before
image

After (notice the link and paragraphs in the description)
image

@KGB33
Copy link
Collaborator

KGB33 commented Mar 28, 2024

Oh, I think the description column in src/web/templates/web/partials/table/table_events.htm can be markdownify-ed too.

@joeriddles joeriddles merged commit ffcdea4 into main Mar 28, 2024
4 checks passed
@joeriddles joeriddles deleted the add-markdown-support branch March 28, 2024 22:24
@joeriddles joeriddles mentioned this pull request Mar 29, 2024
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@KGB33 KGB33 KGB33 approved these changes

@davidslusser davidslusser Awaiting requested review from davidslusser

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@joeriddles @KGB33