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.

Sign up for GitHub

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

Jump to bottom

WIP: Feature/automates resource based on eloquent #2

Open

frapidev wants to merge 2 commits into main from feature/automates-resource-based-on-eloquent

Contributor

frapidev commented Feb 27, 2023 •

edited

Loading

Checkout Laravel Docs before making this feature complete: https://laravel.com/docs/10.x/eloquent-resources. There is a lot of functionality in Laravel already to make this much easier!

Rebase on Refactors test code and fixes duplicate includes issue #1 once it has merged to development
Needs refactor of id
Needs refactor of type
Needs refactor of attributes
Needs refactor of relationships
Needs refactor of meta
Needs refactor of links
Do write test combining resources with different 'filling' methods (auto or register method)
Add check of performance of creating a resource array based on register method vs. based on automated method

frapidev added 2 commits

March 7, 2023 13:40


          Refactors tests to more readable format and fixes include duplicates bug

cdc2dac


          WIP: Improves documentation with test related examples

decc4f0

frapidev force-pushed the feature/automates-resource-based-on-eloquent branch from 521bf62 to decc4f0 Compare

March 7, 2023 12:40

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet