Skip to content

Feat/snapshot source info#604

Open
colpiche wants to merge 3 commits intonicotsx:mainfrom
colpiche:feat/snapshot-source-info
Open

Feat/snapshot source info#604
colpiche wants to merge 3 commits intonicotsx:mainfrom
colpiche:feat/snapshot-source-info

Conversation

@colpiche
Copy link

@colpiche colpiche commented Mar 2, 2026

What

Display snapshot source information (hostname and paths) in the repository snapshots table.

Why

Users had no way to identify where a snapshot originated from directly in the list view,
requiring them to open each snapshot individually.

Changes

Server

  • Fix listSnapshots controller mapping that was silently dropping hostname from the response

Client

  • Add "Source" column to the snapshots table displaying hostname and paths
  • Extend search filter to include hostname
  • Update colSpan accordingly

Testing

  • Manually verified hostname and paths render correctly in the table

@CLAassistant
Copy link

CLAassistant commented Mar 2, 2026

CLA assistant check
All committers have signed the CLA.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 2, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: d21c79f6-c6f2-47cb-9a99-67ac2112bed0

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@nicotsx
Copy link
Owner

nicotsx commented Mar 11, 2026

Hello @colpiche thanks for adding this, however I don't think this UI will work out. It'll be quickly unusable for snapshots that have many include patsh

image

You can open an issue and we'll discuss the best implementation there

@colpiche
Copy link
Author

You're right ! We could start with the hostname only and then discuss the display of folders if relevant.

@nicotsx
Copy link
Owner

nicotsx commented Mar 11, 2026

@colpiche the hostname first sounds good!

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.

3 participants