Re: [Django] #32618: Aggregation documentation should mention conditional aggregation

[Django]

#32618: Aggregation documentation should mention conditional aggregation
-+-
 Reporter:  Simon Willison   |Owner:  Hasan
 Type:   |  Ramezani
  Cleanup/optimization   |   Status:  closed
Component:  Documentation|  Version:  3.2
 Severity:  Normal   |   Resolution:  fixed
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  0
-+-

Comment (by Mariusz Felisiak ):

 In [changeset:"078bcd587dd0b4cc395713be02c38549f5fe65e3" 078bcd58]:
 {{{
 #!CommitTicketReference repository=""
 revision="078bcd587dd0b4cc395713be02c38549f5fe65e3"
 [3.2.x] Fixed #32618 -- Added link to conditional aggregation in
 aggregation topic guide.

 Backport of 1351f2ee163145df2cf5471eb3e57289f8853512 from main
 }}}

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/064.0577bf5812696e8fdbdbb647358436a2%40djangoproject.com.

Re: [Django] #32618: Aggregation documentation should mention conditional aggregation

[Django]

#32618: Aggregation documentation should mention conditional aggregation
-+-
 Reporter:  Simon Willison   |Owner:  Hasan
 Type:   |  Ramezani
  Cleanup/optimization   |   Status:  closed
Component:  Documentation|  Version:  3.2
 Severity:  Normal   |   Resolution:  fixed
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  0
-+-
Changes (by GitHub ):

 * status:  assigned => closed
 * resolution:   => fixed


Comment:

 In [changeset:"1351f2ee163145df2cf5471eb3e57289f8853512" 1351f2e]:
 {{{
 #!CommitTicketReference repository=""
 revision="1351f2ee163145df2cf5471eb3e57289f8853512"
 Fixed #32618 -- Added link to conditional aggregation in aggregation topic
 guide.
 }}}

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/064.4d23736c866d8cc21fc860837ab5f951%40djangoproject.com.

Re: [Django] #32618: Aggregation documentation should mention conditional aggregation

[Django]

#32618: Aggregation documentation should mention conditional aggregation
-+-
 Reporter:  Simon Willison   |Owner:  Hasan
 Type:   |  Ramezani
  Cleanup/optimization   |   Status:  assigned
Component:  Documentation|  Version:  3.2
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  0
-+-
Changes (by Hasan Ramezani):

 * owner:  nobody => Hasan Ramezani
 * status:  new => assigned
 * has_patch:  0 => 1


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/064.162fca31e5780be4016bae8efd1aa66e%40djangoproject.com.

Re: [Django] #32618: Aggregation documentation should mention conditional aggregation

[Django]

#32618: Aggregation documentation should mention conditional aggregation
--+
 Reporter:  Simon Willison|Owner:  nobody
 Type:  Cleanup/optimization  |   Status:  new
Component:  Documentation |  Version:  3.2
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  0 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  0
--+
Changes (by Mariusz Felisiak):

 * needs_docs:  1 => 0
 * type:  Bug => Cleanup/optimization
 * easy:  0 => 1
 * stage:  Unreviewed => Accepted


Comment:

 There is also an example in the
 [https://docs.djangoproject.com/en/3.1/topics/db/aggregation/#cheat-sheet
 Cheat sheet]. A small reference should be enough, e.g.
 {{{
 diff --git a/docs/topics/db/aggregation.txt
 b/docs/topics/db/aggregation.txt
 index cc6310052a..2e882e2a1a 100644
 --- a/docs/topics/db/aggregation.txt
 +++ b/docs/topics/db/aggregation.txt
 @@ -354,7 +354,7 @@ authors with a count of highly rated books::
  >>> Author.objects.annotate(num_books=Count('book'),
 highly_rated_books=highly_rated)

  Each ``Author`` in the result set will have the ``num_books`` and
 -``highly_rated_books`` attributes.
 +``highly_rated_books`` attributes. See also :ref:`conditional-
 aggregation`.

  .. admonition:: Choosing between ``filter`` and ``QuerySet.filter()``
 }}}

 What do you think?

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/064.4b5f2dd3156c9dc662eb56ed6c69a8bd%40djangoproject.com.

Re: [Django] #32618: Aggregation documentation should mention conditional aggregation

[Django]

#32618: Aggregation documentation should mention conditional aggregation
+--
 Reporter:  Simon Willison  |Owner:  nobody
 Type:  Bug |   Status:  new
Component:  Documentation   |  Version:  3.2
 Severity:  Normal  |   Resolution:
 Keywords:  | Triage Stage:  Unreviewed
Has patch:  0   |  Needs documentation:  1
  Needs tests:  0   |  Patch needs improvement:  0
Easy pickings:  0   |UI/UX:  0
+--

Comment (by Simon Willison):

 I found one mention here:
 https://docs.djangoproject.com/en/3.1/topics/db/aggregation/#filtering-on-
 annotations

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/064.e4e7f76b6fc8427183a9402fe9ad3dfc%40djangoproject.com.