Skip to content

ui: move test result and trigger archive to contextual locations#3

Merged
TerrifiedBug merged 1 commit intomainfrom
ui/dashboard-config-tweaks
Feb 27, 2026
Merged

ui: move test result and trigger archive to contextual locations#3
TerrifiedBug merged 1 commit intomainfrom
ui/dashboard-config-tweaks

Conversation

@TerrifiedBug
Copy link
Copy Markdown
Owner

Summary

  • Show connection test result inline next to the Test Connection button instead of at the bottom next to Save
  • Move Trigger Archive button into the top-right of the Last Archive dashboard card

Test plan

  • Verify test connection message appears next to the Test Connection button for both NFS and CIFS
  • Verify save message still appears next to the Save button
  • Verify Trigger Archive button renders in the Last Archive card header

- Show connection test result next to the Test Connection button
  instead of next to the Save button at the bottom
- Move Trigger Archive button into the Last Archive card header
@greptile-apps
Copy link
Copy Markdown

greptile-apps bot commented Feb 27, 2026

Greptile Summary

This PR improves UI feedback by displaying test connection results inline next to the Test Connection button and relocating the Trigger Archive button to the Last Archive card header for better contextual placement.

  • Separated testMessage state from message state in Config.tsx to independently manage test connection feedback
  • Both NFS and CIFS test flows now display results inline next to their respective Test Connection buttons
  • Trigger Archive button moved from standalone section to Last Archive card header with adjusted sizing (text-xs)
  • All functionality preserved - no logic changes, only UI repositioning

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • Clean UI refactoring with no logic changes - only state management separation and element repositioning. All existing functionality preserved, proper React patterns followed, and consistent with codebase styling conventions.
  • No files require special attention

Important Files Changed

Filename Overview
web/src/pages/Config.tsx Separated test connection feedback into dedicated state and displays it inline next to Test Connection button
web/src/pages/Dashboard.tsx Moved Trigger Archive button into Last Archive card header for better contextual placement

Last reviewed commit: fcc5ff7

@TerrifiedBug TerrifiedBug merged commit 7ce0f2a into main Feb 27, 2026
9 checks passed
@TerrifiedBug TerrifiedBug deleted the ui/dashboard-config-tweaks branch February 27, 2026 13:47
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.

1 participant