Skip to content

Item Checkin/Checkout: Fixed #19174 - Allow file uploads - #19189

Open
snipe wants to merge 9 commits into
developfrom
upload-files-on-checkout-and-checkin
Open

Item Checkin/Checkout: Fixed #19174 - Allow file uploads#19189
snipe wants to merge 9 commits into
developfrom
upload-files-on-checkout-and-checkin

Conversation

@snipe

@snipe snipe commented Jun 15, 2026

Copy link
Copy Markdown
Member

This allows a file upload on checkin/checkout for items.

Fixes #19174

@codacy-production

codacy-production Bot commented Jun 15, 2026

Copy link
Copy Markdown

Not up to standards ⛔

🔴 Issues 7 medium

Alerts:
⚠ 7 issues (≤ 0 issues of at least minor severity)

Results:
7 new issues

Category Results
UnusedCode 1 medium
Complexity 6 medium

View in Codacy

🟢 Metrics 22 complexity

Metric Results
Complexity 22

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@snipe

snipe commented Jun 15, 2026

Copy link
Copy Markdown
Member Author

Actually, I found a flaw in the way I've implemented this. Since any user with file management permission can delete files, the files tab would effectively let them delete the checkin/checkout action, which is clearly not good. Refactoring now.

@jaylenkorth-jdmccormick

Copy link
Copy Markdown

Hi @snipe, we're a property management company using Snipe-IT to track vehicles and tools for our maintenance team and this feature would be really valuable for us. Any sense of where it sits in the queue?

@jaylenkorth-jdmccormick

Copy link
Copy Markdown

Hi @snipe, just wanted to follow up since it looks like a new sprint just kicked off. Any chance this gets pulled in? We're actively waiting on it for our maintenance workflow.

@snipe

snipe commented Jul 21, 2026

Copy link
Copy Markdown
Member Author

We haven't really figured out a way to implement this in a way that would work. Right now, if a file upload is associated with a checkin/checkout, deleting the file would delete the checkin/checkout action, which we obviously cannot allow.

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