Skip to content

IVS-132 - Replace download file with delete file#271

Open
rw-bsi wants to merge 2 commits intodevelopmentfrom
IVS-132_Replace_DownloadFile_with_DeleteFile
Open

IVS-132 - Replace download file with delete file#271
rw-bsi wants to merge 2 commits intodevelopmentfrom
IVS-132_Replace_DownloadFile_with_DeleteFile

Conversation

@rw-bsi
Copy link
Contributor

@rw-bsi rw-bsi commented Feb 18, 2026

  • remove Download (UI + backend)
  • remove Revalidate (UI + backend)
  • add 'Delete IFC file'
  • apply same logic for visibility/enabled
  • add confirmation dialogs

Preview:
image

Copy link
Contributor

@Ghesselink Ghesselink left a comment

Choose a reason for hiding this comment

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

Looks good & runs well locally.

I'd say add a functionality to give users a warning when they delete a file? So when they accidentally click it. Should be very quick.

Smth like (untested)

1. Inline delete button — [line 450-453]

// Current:
onClick={ (evt) => { 
  evt.stopPropagation(); 
  setDeleted(row.id);
}}

// Add confirm:
onClick={ (evt) => { 
  evt.stopPropagation(); 
  if (window.confirm('Are you sure you want to delete this IFC file?'))
    setDeleted(row.id);
}}
2. Toolbar bulk delete — [line 336-338]


// Current:
function onDelete() {
  setDeleted(selected.join(','))
}

// Add confirm:
function onDelete() {
  if (window.confirm(`Delete ${selected.length} selected file(s)?`))
    setDeleted(selected.join(','))
}

@rw-bsi rw-bsi deployed to development February 18, 2026 22:43 — with GitHub Actions Active
@rw-bsi rw-bsi requested a review from Ghesselink February 18, 2026 22:43
@rw-bsi rw-bsi self-assigned this Feb 18, 2026
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

Comments