Skip to content

Conversation

cliff688
Copy link
Member

@cliff688 cliff688 commented Apr 16, 2025

ticket-36329

Removes non-code custom link text used for cross-referencing Python objects in the documentation, so that they are not rendered as inline code.

e.g. :meth:`a get() lookup <django.db.models.QuerySet.get>`

GitHub regex search:

repo:django/django /:(?:class|meth|func|attr):`\s*\w+(?:\s+\w+)+\s*</ path:docs/

@cliff688 cliff688 force-pushed the ticket-36329 branch 3 times, most recently from 6898bdd to 235ba38 Compare April 17, 2025 04:40
Copy link
Contributor

@sarahboyce sarahboyce left a comment

Choose a reason for hiding this comment

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

Thank you! Added initial comments ⭐

@cliff688 cliff688 force-pushed the ticket-36329 branch 2 times, most recently from bf172ca to cd60a67 Compare May 21, 2025 10:57
Copy link
Contributor

@sarahboyce sarahboyce left a comment

Choose a reason for hiding this comment

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

I think we might also want to change docs/faq/intall.txt:

:djadmin:`lightweight development server<runserver>`

Thank you for this! I think this is really close 👍

Comment on lines -475 to +478
When removing different pairs of objects from :class:`ManyToManyFields
<django.db.models.ManyToManyField>`, use
:meth:`~django.db.models.query.QuerySet.delete` on a
:class:`~django.db.models.Q` expression with multiple
:attr:`~django.db.models.ManyToManyField.through` model instances to reduce
the number of SQL queries. For example::
When removing multiple many-to-many relationships involving different objects
on both sides, use the :meth:`~django.db.models.query.QuerySet.delete` method
on a filtered queryset of the field’s
:attr:`~django.db.models.ManyToManyField.through` model. By combining multiple
conditions using :ref:`q-objects`, you can remove several relationships in a
single query. For example::
Copy link
Contributor

@sarahboyce sarahboyce Jul 24, 2025

Choose a reason for hiding this comment

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

This has been rephrased quite a bit, I need to re-read (marking this as a TODO for me)

…ing Python objects.

Thanks Bruno Alla and Sarah Boyce for reviews.

Co-authored-by: Sarah Boyce <[email protected]>
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.

3 participants