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

Refs #28592 -- Improved some headings in CSRF how-to. #15531

Merged
merged 1 commit into from Mar 22, 2022
Merged

Conversation

tommcn
Copy link
Contributor

@tommcn tommcn commented Mar 21, 2022

Ticket https://code.djangoproject.com/ticket/28592
Inspired by PR #9071 but with the conflicts resolved.

Part of a series of PRs as suggested in #14956.

  • Update headings in howto/csrf to make them more consistent

@felixxm felixxm changed the title Refs #28592 -- Made CSRF how-to headings more consistent Refs #28592 -- Improved some headings in CSRF how-to. Mar 22, 2022
@felixxm felixxm merged commit f77216b into django:main Mar 22, 2022
@felixxm
Copy link
Member

felixxm commented Mar 22, 2022

@tommcn Thanks 👍

Setting the token when CsrfViewMiddleware.process_view is not used
------------------------------------------------------------------
Setting the token when ``CsrfViewMiddleware.process_view()`` is not used
------------------------------------------------------------------------
Copy link
Member

Choose a reason for hiding this comment

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

Can you confirm you are happy this is a --- level heading and not a ===.

Copy link
Member

Choose a reason for hiding this comment

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

Yes, it's in Edge cases as before.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants