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.

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

Fixed #34595 -- Doc'd that format_string arg to format_html() is not escaped #16913

Merged
merged 1 commit into from Jun 1, 2023

Conversation

apjama
Copy link
Contributor

@apjama apjama commented Jun 1, 2023

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hello! Thank you for your contribution 💪

As it's your first contribution be sure to check out the patch review checklist.

If you're fixing a ticket from Trac make sure to set the "Has patch" flag and include a link to this PR in the ticket!

If you have any design or process questions then you can ask in the Django forum.

Welcome aboard ⛵️!

escaped but all other args and kwargs are passed through
:func:`conditional_escape` before being passed to ``str.format()``.
Finally, The output has :func:`~django.utils.safestring.mark_safe` applied.

Copy link
Member

Choose a reason for hiding this comment

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

Chop blank line.

docs/ref/utils.txt Outdated Show resolved Hide resolved
@smithdc1
Copy link
Member

smithdc1 commented Jun 1, 2023

Hey -- thanks for the PR.

Likely the commit message should be something like

Fixed #34595 -- Doc'd that format_string arg to format_html() is not escaped

@apjama apjama changed the title Fixed #34595 format_html() docs should explicitly mention that format_string is not escaped Fixed #34595 -- Doc'd that format_string arg to format_html() is not escaped Jun 1, 2023
@felixxm
Copy link
Member

felixxm commented Jun 1, 2023

@apjama Thanks for the patch 👍 Welcome aboard ⛵

@felixxm felixxm merged commit 4037223 into django:main Jun 1, 2023
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants