Reversed order of arguments in the reverse order of Ecto

[tadiou]

the keyword args of desc: :id vs id: :desc. the former being ecto, the latter being paginator.

Is there a reason why they're incongruent, and if not, would y'all accept a PR to switch that to be congruent to what Ecto is currently doing?

[sgerrand]

Hello and thanks for your question. For the my benefit and that of other future readers, would you please expand on what you have referred to in your question, ideally with a working example?

[glennr]

In Ecto:

Ecto.Query.order_by(query, [desc: :inserted_at, desc: :id])

With paginator, the cursor arguments have these reversed:

  paginate(query, cursor_fields: [inserted_at: :desc, id: :desc])

Given query sorting and cursor generation are so closely related, it'd be nice to be able to use the same arguments, based on how Ecto does it i.e. [desc: :inserted_at, desc: :id]