Skip to content

fix: escape headers and unordered lists in markdown utils#1058

Open
onerandomusername wants to merge 7 commits into
masterfrom
expanded-markdown
Open

fix: escape headers and unordered lists in markdown utils#1058
onerandomusername wants to merge 7 commits into
masterfrom
expanded-markdown

Conversation

@onerandomusername

Copy link
Copy Markdown
Member

Summary

Expanded markdown support in remove markdown and escape markdown methods. This does not currently support ordered lists, unfortunately.

Checklist

  • If code changes were made, then they have been tested
    • I have updated the documentation to reflect the changes
    • I have formatted the code properly by running pdm lint
    • I have type-checked the code by running pdm pyright
  • This PR fixes an issue
  • This PR adds something new (e.g. new method or parameters)
  • This PR is a breaking change (e.g. methods or parameters removed/renamed)
  • This PR is not a code change (e.g. documentation, README, ...)

@onerandomusername onerandomusername added t: api support Support of Discord API features s: needs review Issue/PR is awaiting reviews labels Jun 19, 2023
@onerandomusername onerandomusername added this to the disnake v2.9 milestone Jun 19, 2023

@shiftinv shiftinv left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This replaces # and - anywhere in a string, not only at the start of a line

Seems like something that could use a bunch of testing, so moving it to the next version might make sense here.

@onerandomusername

Copy link
Copy Markdown
Member Author

This replaces # and - anywhere in a string, not only at the start of a line

Seems like something that could use a bunch of testing, so moving it to the next version might make sense here.

Sigh, that's fair. Possibly can be part of a bugfix release, like 2.9.1

@onerandomusername

Copy link
Copy Markdown
Member Author

Due to more quirks with the client this entire implementation is wrong.

@shiftinv shiftinv added s: planned Issue is planned in the future t: bugfix and removed s: needs review Issue/PR is awaiting reviews labels Dec 26, 2024
@shiftinv shiftinv removed this from the disnake v2.10 milestone Dec 26, 2024
@shiftinv shiftinv removed the s: planned Issue is planned in the future label Jul 11, 2026
@read-the-docs-community

Copy link
Copy Markdown

Documentation build overview

📚 disnake | 🛠️ Build #33546276 | 📁 Comparing c1df806 against latest (863033f)

  🔍 Preview build  

48 files changed · ± 48 modified

± Modified

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t: api support Support of Discord API features t: bugfix

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

2 participants