Skip to content
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

I can sort processes and process instances #293

Merged
merged 4 commits into from
Sep 3, 2021
Merged

I can sort processes and process instances #293

merged 4 commits into from
Sep 3, 2021

Conversation

saig0
Copy link
Contributor

@saig0 saig0 commented Sep 3, 2021

  • show the deployment time of processes
  • add sorting of processes by deployment time
  • add sorting of process instances by start time

closes #275

* in the process view, allow sorting by start time
* make the pagination aware of the sorting
* in the processes view, allow sorting by deployment time
* in the instances view, allow sorting by start time
@saig0 saig0 merged commit 6273e1d into master Sep 3, 2021
@saig0 saig0 deleted the 275-sorting branch September 3, 2021 12:19
@danshapir
Copy link

@saig0 is the sorting done on the client side? I dont see any backend logic here.

@saig0
Copy link
Contributor Author

saig0 commented Sep 6, 2021

@danshapir no, the sorting happens in the backend 😄

It is a built-in feature of Spring data web. Every repository supports pagination and sorting by default. So, the client just need to add the sorting parameter 🚀

I build the sorting in a way that is easy to adapt for other tables. See here:

https://github.com/camunda-community-hub/zeebe-simple-monitor/pull/293/files#diff-dc86492ab8732f0eeedb88ff1c1074e9024959f535dc8cda0dcf2c3d0607626d

@danshapir
Copy link

Awesome!

@danshapir
Copy link

@saig0 can you release a new docker version?

@saig0
Copy link
Contributor Author

saig0 commented Sep 8, 2021

@danshapir there is also a Docker image. You can get it here:

docker pull ghcr.io/camunda-community-hub/zeebe-simple-monitor:2.1.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sorting - everything is sorted ASC and not DESC
2 participants