Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed checkout_date for licenses on print all assigned for users #14407

Merged
merged 2 commits into from
Mar 10, 2024

Conversation

snipe
Copy link
Owner

@snipe snipe commented Mar 10, 2024

This fixes an issue where the updated_at date was not being included in the user->license relationship, so the checkout date was not being included in the "print all assigned". This acts a little different because of the (dumb) way we handle the license joins right now, where the join entry gets created, but it actually matters when it's updated, since the join record can exist unassigned.

Copy link

Copy link

what-the-diff bot commented Mar 10, 2024

PR Summary

  • Addition of Timestamps to 'licenses' method in the 'User' Model
    The method for handling licenses within the User model has been enhanced. It now includes 'created_at' and 'updated_at' timestamps. These fields will provide more information about when the license was created and last updated.

  • Display of 'updated_at' Timestamp in 'print.blade.php' View
    The 'updated_at' timestamp from the 'pivot' table is now shown in the 'print.blade.php' view for each license. This allows users to easily see when a specific license was last updated.

@snipe snipe merged commit 1df56a7 into develop Mar 10, 2024
8 checks passed
@snipe snipe deleted the bug/sc-25019_added_created_at_pivot branch March 10, 2024 16:40
@snipe snipe changed the title Added created_at to license pivot Added updated_at to license pivot Mar 10, 2024
@snipe snipe changed the title Added updated_at to license pivot Fixed checkout_date for licenses on print all assigned for users Mar 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant