Skip to content

servershell: support multiple objects in history and changes - #478

Merged
brainexe merged 3 commits into
mainfrom
md_multiple_history
Oct 22, 2026
Merged

brainexe merged 3 commits into
mainfrom
md_multiple_history

Conversation

@brainexe

@brainexe brainexe commented Oct 9, 2026 •

Copy link
Copy Markdown
Member
  • history view accepts multiple object_id/hostname params. e.g. get the time for memory or other changes for a bunch of servers.
  • add hostname column linking to inspect page
  • fix XSS

@kofrezo kofrezo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I personally don't like it too much. I think it bloats the already bloated history view and makes it harder to read. Changes is fine as it does not change the display much. But if people find it useful I won't complain.

But lets at least tackle the code changes.

@brainexe brainexe changed the title servershell: support multiple objects in history and changes DRAFT: servershell: support multiple objects in history and changes Oct 12, 2026
@kofrezo
kofrezo marked this pull request as draft October 12, 2026 14:19
@kofrezo kofrezo changed the title DRAFT: servershell: support multiple objects in history and changes servershell: support multiple objects in history and changes Oct 12, 2026
@brainexe brainexe closed this Oct 21, 2026
@brainexe brainexe reopened this Oct 21, 2026
@brainexe
brainexe marked this pull request as ready for review October 21, 2026 04:44
- history view accepts multiple object_id/hostname params
- add hostname column linking to inspect page
- changes command/view use all selected objects
- fix empty "Time Since" and recreate object_id on deleted rows
@brainexe
brainexe force-pushed the md_multiple_history branch from 0c81fd9 to 680f271 Compare October 21, 2026 05:04
@brainexe
brainexe merged commit 283c4af into main Oct 22, 2026
5 checks passed
@kofrezo
kofrezo deleted the md_multiple_history branch October 22, 2026 05:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants