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

UI - Fixes #2 #168

Merged
merged 13 commits into from
Aug 3, 2023
Merged

UI - Fixes #2 #168

merged 13 commits into from
Aug 3, 2023

Conversation

wcgunter
Copy link
Collaborator

@wcgunter wcgunter commented Aug 3, 2023

This PR changes the following...

  • Logs: The columns can now be ordered (sorted in ascending / descending).
    image

  • Deployments: A new column has been added to the start of the table so that the "Name" column header is above the name of the process definition, not the icons.

image
  • Processes: Updated wording of "Delete selected rows" to "Stop running selected rows"
image
  • History: The output variables table is now combined with the existing information table. Note has been added clarifying that all output variables start with "output_" even though they are not displayed that way.
image
  • History: Removed "show all" from output variables list and added support for "output_display_order", an array of output variable names. The UI will try to display the variables in that order. If a variable listed cannot be found, it is skipped. If a variable is not listed, it gets appended to the end in the order it is read.
image

@wcgunter wcgunter marked this pull request as ready for review August 3, 2023 18:35
@jamesfwood jamesfwood self-requested a review August 3, 2023 18:39
Copy link
Collaborator

@jamesfwood jamesfwood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work!

@jamesfwood jamesfwood merged commit 6c175c7 into develop Aug 3, 2023
@jamesfwood jamesfwood deleted the US-misc-2 branch August 3, 2023 18:40
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