-
Notifications
You must be signed in to change notification settings - Fork 0
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
Updated Deliver Status View #297
Conversation
4520966
to
751646a
Compare
4516371
to
f1483fd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left a small comment that I thought I had made weeks ago, otherwise seems fine to me.
@@ -81,4 +81,5 @@ def get_annotated_opportunity_access_deliver_status(opportunity: Opportunity): | |||
) | |||
.order_by("user__name") | |||
) | |||
access_objects.sort(key=lambda a: a.user.name) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is fine if necessary, but is it possible sort in sql?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried to do the same query using SQL, to keep the sorting in SQL but ran into issues with payment units not showing up for users who have not submitted any forms, as no completed works were created.
Ticket