Skip to content

Conversation

@Godmartinz
Copy link
Member

@Godmartinz Godmartinz commented Oct 23, 2025

This refactors the getExpiringWarrantyOrEol() method without using whereRaw(). By breaking up the $expired_assets and $assets_with_warranties, we can manipulate the $expiration_window search on warranties easier using betweenIncluded() instead of whereRaw()

image

@Godmartinz Godmartinz requested a review from snipe as a code owner October 23, 2025 18:13
@snipe snipe requested a review from uberbrady October 25, 2025 14:03
@snipe
Copy link
Member

snipe commented Oct 25, 2025

If the warrantees are already expired, I wouldn't think we'd still want them in the alerts, otherwise that list could end up getting very long, no?

@Godmartinz
Copy link
Member Author

I meant this returns expiring warranties within the X days window given from the notification settings.

@Godmartinz
Copy link
Member Author

I will rename that variable as well 😅

@Godmartinz Godmartinz changed the title Fixed expired warranties not being included in the expiring alerts notification Fixed expiring warranties not being included in the expiring alerts notification Oct 26, 2025
@snipe
Copy link
Member

snipe commented Oct 27, 2025

Does this fix a particular GH issue?

@snipe snipe merged commit 06eab5f into grokability:develop Oct 27, 2025
8 checks passed
@Godmartinz Godmartinz linked an issue Oct 27, 2025 that may be closed by this pull request
6 tasks
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.

[Bug]: expiring-alerts shows 0 expiring assets in 8.3.2 and 8.3.3

2 participants