- fix lazy loading in VacationRequest model

[mtracz]

This PR fixes lazy loading user relation in VacationRequest model in queued jobs.

Sentry reports:

Illuminate\Database\LazyLoadingViolationException
Attempted to lazy load [user] on model [Toby\Models\VacationRequest] but lazy loading is disabled.

• https://blumilk.sentry.io/issues/5139914913/?project=4505271823237120&query=is%3Aunresolved&referrer=issue-stream&statsPeriod=30d&stream_index=0
• https://blumilk.sentry.io/issues/5139914961/?project=4505271823237120&query=is%3Aunresolved&referrer=issue-stream&statsPeriod=30d&stream_index=1

Places where user relation is lazy loaded

• https://github.com/blumilksoftware/toby/blob/main/app/Slack/Elements/VacationRequestsAttachment.php#L33
• https://github.com/blumilksoftware/toby/blob/main/app/Notifications/VacationRequestsSummaryNotification.php#L70

---

Update:

Laravel is able to restore single model relations in queued job during __unserialize(). Relations in the collection are not restored by design, see:

• https://github.com/laravel/framework/issues/36824

https://github.com/laravel/framework/blob/11.x/src/Illuminate/Queue/SerializesAndRestoresModelIdentifiers.php#L61-L62

Relations restored in single model: https://github.com/laravel/framework/blob/11.x/src/Illuminate/Queue/SerializesAndRestoresModelIdentifiers.php#L109

Relations not restored in collection: https://github.com/laravel/framework/blob/11.x/src/Illuminate/Queue/SerializesAndRestoresModelIdentifiers.php#L81

In this case we need to load required relations manually in queued job after it is unserialized from the queue. Due to each channel (toMail(), toSlack()) is a separate job, we have to load relations for each one.

Tested locally and it works.

[mtracz]

Code owner review is required.