Skip to content

Conversation

akemidx
Copy link
Member

@akemidx akemidx commented Aug 12, 2025

Adding in an option to log an audit when you check in or out an asset.
Screenshot 2025-04-09 at 7 29 22 PM

Screenshot 2025-04-09 at 7 30 02 PM

There is less information on the audit itself, but all changes to an asset will be logged on the checkin or out.

NOTE: This action does not allow a user to manually set a next audit date OR upload a picture as these are not options on checkin or checkout

Update of #16679 after a bad merge.

@akemidx akemidx requested a review from snipe as a code owner August 12, 2025 22:11
@akemidx
Copy link
Member Author

akemidx commented Aug 12, 2025

I am currently working on a derivitive of this that will allow the addition of setting the next audit date or other actions that can be taken on a normal audit.

@akemidx
Copy link
Member Author

akemidx commented Aug 12, 2025

Oh additionally, notes get double logged I think. This is a current code limitation issue where all of our notes are handled the same, whether its an audit or a checkout, we just call it note

@snipe snipe changed the title Looging Audit on Checkin and Checkout Loging Audit on Checkin and Checkout Aug 15, 2025
@snipe snipe changed the title Loging Audit on Checkin and Checkout Logging Audit on Checkin and Checkout Aug 15, 2025
@snipe
Copy link
Member

snipe commented Aug 15, 2025

Should this be marked as a draft/WIP then?

@snipe snipe marked this pull request as draft August 18, 2025 01:46
@akemidx
Copy link
Member Author

akemidx commented Aug 18, 2025

Should this be marked as a draft/WIP then?

So, we had a discussion a bit ago about just getting it to log the audit. Which is where this PR comes from.

However, looking forward, and wanting to have some sort of utility with it, I made another branch to add those extra features on. We can draft this if we just wanna ship all at once, but since the original just needed to be caught up, and the merge borked, I slung this again without those extra bits.

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