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

Added PHPDocs for create, update, delete, all, retrieve methods after moving them out of traits. #1701

Merged
merged 3 commits into from
May 30, 2024

Conversation

prathmesh-stripe
Copy link
Contributor

@prathmesh-stripe prathmesh-stripe commented May 28, 2024

Moved Create, Update, Delete, All, Retrieve traits to resources.

@prathmesh-stripe prathmesh-stripe self-assigned this May 28, 2024
@ramya-stripe
Copy link
Contributor

Are we not planning to delete the files that define the different traits that we are now not using?

@ramya-stripe
Copy link
Contributor

And are we not planning to remove the update method from the Update trait?

@prathmesh-stripe
Copy link
Contributor Author

We can't remove the traits as there is a concern with these overrides that still use the traits.

I'll have to look into these and change some behavior(or the override itself).

@ramya-stripe ramya-stripe changed the title Demagiced crudl operation Moved methods for create, update, delete, all & retrieve operations out of traits May 30, 2024
@ramya-stripe ramya-stripe changed the title Moved methods for create, update, delete, all & retrieve operations out of traits Added PHPDocs for create, update, delete, all, retrieve methods after moving them out of traits. May 30, 2024
@prathmesh-stripe prathmesh-stripe merged commit 92dab43 into master May 30, 2024
22 checks passed
@prathmesh-stripe prathmesh-stripe deleted the prathmesh_php_demagicing_crudl branch May 30, 2024 20:07
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.

3 participants