snipe comments

Results 1045 comments of


                                            snipe

Added Daily depreciation calculations for assets and reports

Just my question above: `Will this affect the other select2's throughout the system as well?`

Added Daily depreciation calculations for assets and reports

Ping @Godmartinz

Adds require serial as Asset Model option

@viclou if there was, you’d have seen it here.

Adds require serial as Asset Model option

@DECStudioHub - this PR *is* the solution. It's just still in testing.

Adds require serial as Asset Model option

@Godmartinz did we get the requiredness indicator handled?

Adds require serial as Asset Model option

@DECStudioHub If you select that on the model, yes, per the original PR summary.

WIP: Seperated Notifications and Emails: Check ins and Check outs

@Godmartinz I honestly don't remember. I don't think so, but you'd have to look at the existing code.

WIP: Seperated Notifications and Emails: Check ins and Check outs

@Godmartinz isn't this actually initiating the send tho? https://github.com/snipe/snipe-it/pull/15681/files#diff-b463d9fd627c8705fc22d512ea776cdec9cdddf67f476c26acd28ed0c40f0de6R79 GitHubSeparated Notifications and Emails: Check ins and Check outs by Godmartinz · Pull Request #15681 · snipe/snipe-itDescription This separates Check in/out...

WIP: Seperated Notifications and Emails: Check ins and Check outs

I mean, it's easy enough to test. Change your email notification and/or webhook URL to something that isn't valid (directly in the DB so you skip validation) and tail the...

Fixes Assets location not being resynced when Users location is updated via LDAP

This looks good, @Godmartinz - can you just handle that one small conflict that popped up?