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

SK Server Admin UI Activity View Update #1669

Merged
merged 13 commits into from
Feb 11, 2024

Conversation

KEGustafsson
Copy link
Contributor

@KEGustafsson KEGustafsson commented Feb 3, 2024

This PR introduces changes to Connection activity view on SK Admin UI.
Currently activity view is mixed list of Connections and Plugins activities and list is not sorted at all.
Update will split activities to Connections and Plugins and sort list in alphabetic order.

Example view:
image

@tkurki tkurki added the feature label Feb 4, 2024
@tkurki
Copy link
Member

tkurki commented Feb 4, 2024

The headings could be "heavier", so that you can tell that they are headings for the list below.

Don't know how relevant in real life, but if there's no plugin activity the heading looks a little oprhaned

image

@tkurki
Copy link
Member

tkurki commented Feb 4, 2024

Minor nuisance, but now all Connections link to Connections page, even if just some of them can be configured there.

@KEGustafsson
Copy link
Contributor Author

Pretty much all fixed now from activity view.

@tedenda
Copy link

tedenda commented Feb 5, 2024

Could #1146 also be looked at in this PR?

@tkurki
Copy link
Member

tkurki commented Feb 5, 2024

Related but independent, better kep separate. But I hear you…

@KEGustafsson
Copy link
Contributor Author

Duplicated/similar code parts are removed.

@KEGustafsson
Copy link
Contributor Author

Fixes and optimizations for Dashboard starts to be finalized for now.

If still something need to be changed or added ok, otherwise ready for review.

@tkurki tkurki merged commit bcbc19a into SignalK:master Feb 11, 2024
4 checks passed
@KEGustafsson KEGustafsson deleted the connections_plugins_ui branch February 12, 2024 21:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants