Pulp

I'm glad you posted this, and I have some general frustration about it.

On the Task model serializer, we have the following fields:

started_at
finished_at
{...}

And the Task viewset filter params are:

started_after
started_before
finished_after
finished_before
   {...}

If you look at a Task record, you might expect that you could perform a query such as "/tasks/?started_at__gte=..." but you cannot, because we've opted for entirely different names: "started_before", "finished_after", etc.

Likewise, on the RepositoryVersion serializer:

number
created
{...}

But on the viewset, the filter params:

version_min
version_max
created_after
created_before
{...}

Same here.

I don't like the inconsistency between the parameters we're providing to filter by and the actual names of the fields. I propose to delete "*_after" and "*_before" filters entirely, in favor of using normal conventions like e.g. "/tasks/?finished_at__lte=..."

That seems straightforward and unambiguous to me. +1.

+1 on this resolution.