Skip to content

Adds #17619 checkouts of an asset to 'All Assigned Email' of a user#18329

Open
Godmartinz wants to merge 3 commits intogrokability:developfrom
Godmartinz:include-asset-assignments-to-email-user-all
Open

Adds #17619 checkouts of an asset to 'All Assigned Email' of a user#18329
Godmartinz wants to merge 3 commits intogrokability:developfrom
Godmartinz:include-asset-assignments-to-email-user-all

Conversation

@Godmartinz
Copy link
Member

@Godmartinz Godmartinz commented Dec 10, 2025

This adds all Assets/Components/Accessories/Licenses assigned to Asset_X, to be displayed for the User of Asset_X in the All Assigned To User Email notification.
Adds #17619

Asset 534683216 has one of each checked out to it.
image
And the User has Asset 534683216, one license, and one accessory checked out for correlation:
image

A total of seven items are now accounted for in the All Assigned Email:
image

@Godmartinz Godmartinz requested a review from snipe as a code owner December 10, 2025 02:05
@Godmartinz Godmartinz marked this pull request as draft December 10, 2025 02:07
@Godmartinz Godmartinz marked this pull request as ready for review December 10, 2025 02:08
@Godmartinz Godmartinz changed the title Adds checkouts of an asset to 'All Assigned email' of a user Adds #17619 checkouts of an asset to 'All Assigned email' of a user Dec 10, 2025
@Godmartinz Godmartinz changed the title Adds #17619 checkouts of an asset to 'All Assigned email' of a user Adds #17619 checkouts of an asset to 'All Assigned Email' of a user Dec 10, 2025
@swift2512
Copy link

@Godmartinz could Custom fields be added to this 'Print all assigned'/'Email List of All Assigned' lists?
Probably, since day one 'Print all assigned'/'Email List of All Assigned' offers less columns than are in 'Your Profile > Accept Items' View. It would be great to have 'Custom Fields'.

@snipe
Copy link
Member

snipe commented Dec 11, 2025

@swift2512 there is a separate PR for that. #17426

@snipe snipe requested a review from bzeus December 11, 2025 16:47
@snipe
Copy link
Member

snipe commented Jan 5, 2026

I think this could get a little confusing though, no? I think we'd probably want to break those out into different sections within the email. Since a component can't be assigned to a user, the report isn't exactly correct the way it is.

Copy link
Member

@snipe snipe left a comment

Choose a reason for hiding this comment

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

I think we should break these out into their own sections in the emailed report - and whichever way we do it, it should be consistent with the way it appears in the "print all assigned" page.

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.

[Feature Request] Include accessories etc checked out to assets that are checked out to the user [recursive] to “email all assigned”

3 participants