cortezaproject / corteza

Low-code platform
https://cortezaproject.org
Apache License 2.0
1.62k stars 370 forks source link

Sorting resource by "last-change" does not work as expected #468

Closed darh closed 1 year ago

darh commented 1 year ago

As reported by @Bojan-Svirkov

I just came a cross this case, so when you are for example in the namespace list in compose and you sort them by last change, the order is mixed up and the namespaces are not sorted in the correct order. Same thing for workflows, when I sort them by Updated at you can see from the video that the order is mixed up, we have entries one by one September, October and then August which cannot be, they need to be in the correct order one by one. Currently I just noticed this on Compose and Workflows but it is possible to be present also on other places. I see that some of the formats where changed from 11/09/2022, to Wednesday, November 9, 2022 at 7:01 AM, so this could be also the mix up. Also from the screenshot you can see that the order is mixed up.

Image


@Fajfa

we sort by updatedAt but show either updatedAt || createdAt

If we want a proper solution we need to discuss it a bit, so next sprint.


Bojan-Svirkov commented 1 year ago

There are still some issues on this. Waiting on Vivek to address them.

Bojan-Svirkov commented 1 year ago

When you sort by last change, the view is kinda broken, meaning always the same results are shown even if you press next page. The total record count is also surpassing the total records we have if we continuously press next. @vicpatel please check this.

https://user-images.githubusercontent.com/53232926/204781639-74e6be1e-360a-497e-92b6-6eec581af252.webm

Bojan-Svirkov commented 1 year ago

@vicpatel Now everything looks good and the only part left is the Changelog entry. Please include it. Thanks.

Fajfa commented 1 year ago

What was fixed:

How was it fixed:

vicpatel commented 1 year ago

Added#

What was added:

Why was it added:

Bojan-Svirkov commented 1 year ago

Added CL entry. @Fajfa & @vicpatel you can close this issue.