Skip to content

Conversation

snipe
Copy link
Member

@snipe snipe commented Jun 9, 2025

This is still broken, but since it came up again recently, I figured I'd push up what I've already done.

@snipe snipe moved this to In progress in Snipe-IT Awaiting Epic Jun 9, 2025
@snipe snipe requested a review from uberbrady June 9, 2025 19:29
@snipe snipe marked this pull request as draft June 9, 2025 19:29
@Toreg87
Copy link
Contributor

Toreg87 commented Jun 11, 2025

@snipe if I understand this correctly this is something I implemented a few years ago: Toreg87@d997a8b
This was rejected upstream but if this seems useful now I can create a new PR. It is currently based on v7 and I am in the process of rebasing it to v8. The scoped locations feature complicates this a bit. I use this in my production site without problems. It could use a little bit love on the UI and is lacking a few translations. But other than that I am quite happy with it.

@snipe
Copy link
Member Author

snipe commented Jun 11, 2025

@Toreg87 I mean, sure! If you can rebase against current v8 and spare me the sorrow, I'd happily take a fresh look at it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In progress

Development

Successfully merging this pull request may close these issues.

2 participants