Skip to content

fix: remove markdownify from bestiary builder trait descriptions#2222

Merged
codybanman merged 1 commit intonightlyfrom
fix-bestiary-builder-markdownify
Oct 10, 2025
Merged

fix: remove markdownify from bestiary builder trait descriptions#2222
codybanman merged 1 commit intonightlyfrom
fix-bestiary-builder-markdownify

Conversation

@codybanman
Copy link
Copy Markdown
Contributor

Summary

Removes unnecessary markdownify() call from Bestiary Builder trait description parsing. Bestiary Builder returns plain text/markdown, while CritterDB returns HTML. This fix ensures trait descriptions from Bestiary Builder are properly displayed without double-processing.

Changelog Entry

Fixed Bestiary Builder trait descriptions being incorrectly processed through markdownify

Checklist

PR Type

  • This PR is a code change that implements a feature request.
  • This PR fixes an issue.
  • This PR adds a new feature that is not an open feature request.
  • This PR is not a code change (e.g. documentation, README, ...)

Other

  • This PR is a breaking change (e.g. methods or parameters removed/renamed)
  • If code changes were made then they have been tested.
  • I have updated the documentation to reflect the changes.

Copy link
Copy Markdown
Contributor

@tiberius0954 tiberius0954 left a comment

Choose a reason for hiding this comment

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

LGTM!

@codybanman codybanman merged commit b262e67 into nightly Oct 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants