Api groups order

[billmn]

Issue: if the request use “?sort=order” returns groups ordered by “order DESC”.

This is unexpected.
If I would a reversed order I have to use “sort=order&order=desc”

[jbrooksuk]

One thing we should add is multiple sorts. Fancy this @billmn?

[billmn]

@jbrooksuk Multiple sorts can introduce some BC but I think we can remove the "order" parameter using only "sort":

?sort=-status,order == ORDER BY status DESC, ORDER ASC

[jbrooksuk]

I like it, but this breaks BC and I don't like that.

[ConnorVG]

I agree with what @jbrooksuk said but I also have a question... What would be the 'default' definition for the sort orders if you're wanting it to go this way? Seems both opinionated and random to me.

[billmn]

@jbrooksuk I agree too, nobody loves BC :)

@ConnorVG If I don't specify the "sort" parameter I use a default sorting (eg: for component groups -> status DESC, order ASC).

If I specify the "sort" parameter:

• with the hyphen the order is descending
• without the hyphen the order is ascending

[ConnorVG]

I see, I didn't notice the hyphen. Thanks.

[jbrooksuk]

I guess we have to break BC on something like this?

[ConnorVG]

We don't have to, especially if we use a versioned API.

[jbrooksuk]

True. Although that's a ball ache to maintain.

[ConnorVG]

🤔 💭