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

Switch from deprecated spatie/data-transfer-object to spatie/laravel-data #571

Merged
merged 2 commits into from
Jul 5, 2023

Conversation

dsindrilaru
Copy link
Contributor

No description provided.

@dsindrilaru dsindrilaru requested a review from binaryk July 5, 2023 08:03
@what-the-diff
Copy link

what-the-diff bot commented Jul 5, 2023

PR Summary

  • New Package Addition
    The spatie/laravel-data package has been added to our project dependencies. This will provide us with additional features and functionality.

  • Updated Method in Action.php
    Modifications were made to the handle method in Action.php. This should improve how the method works, potentially enhancing performance and functionality.

  • Renaming and Updating of Classes
    The classes AdvancedFilterPayloadDto and PaginationDto have been renamed to AdvancedFilterPayloadDataObject and PaginationDataObject respectively. This should make their purpose clearer and more understandable.

  • Addition of New Classes
    New classes, AdvancedFilterPayloadDataObject and PaginationDataObject, were added. These should add new capabilities to our project and improve overall performance and functionality.

  • Adjusted Class References
    References to the renamed classes (AdvancedFilterPayloadDto and PaginationDto) in the AdvancedFilter, AdvancedFiltersCollection, and RestifyRequest classes have been correctly updated. This will avoid any potential issues and confusion caused by the renaming.

@binaryk binaryk merged commit 7fbbe4f into BinarCode:8.x Jul 5, 2023
4 of 8 checks passed
@binaryk
Copy link
Collaborator

binaryk commented Jul 5, 2023

Thank you!

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.

2 participants