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

feat(ui): show only latest processes in main page #773

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bespaltovyj
Copy link

@bespaltovyj bespaltovyj commented Jan 26, 2025

In process page can show only latest process version(#772)

what problem you try to solve?

Processes are constantly changing and old versions are no longer relevant.
I think it is useful to have the ability to turn off the display of old versions

what reasoning you had with this implementation? what alternative approaches exists? what are possible downsides?

I was guided by the old code and made the new logic similar to it

how have you tested your implementation?

I ran the app locally with fake data in the DB and I tested different combinations of filters on this page and everything worked as expected.

New filters:

main

Before creating a PR, please ensure...

  • the code is proper formatted (e.g. running mvn com.spotify.fmt:fmt-maven-plugin:2.25:format)
  • your code contribution contains new unit tests (if applicable)
  • the tests are alle 'green'
  • the documentation is updated (if applicable)

* show latest processes in a list view by filter
@bespaltovyj bespaltovyj force-pushed the feat/filter-process-defenition branch from 3c45909 to f9cdfc9 Compare January 26, 2025 11:53
@bespaltovyj
Copy link
Author

@nitram509 Hello, Martin!. Can you check this PR?

@nitram509
Copy link
Collaborator

Will have a look by next, since my spare time this week is very limited.

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.

2 participants