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

Implementing repository interface is not mandatory anymore #875

Draft
wants to merge 1 commit into
base: 1.11
Choose a base branch
from

Conversation

loic425
Copy link
Member

@loic425 loic425 commented May 17, 2024

Q A
Bug fix? yes
New feature? not really
BC breaks? no
Deprecations? no
Related tickets
License MIT

@loic425 loic425 requested a review from a team as a code owner May 17, 2024 08:17
@loic425 loic425 marked this pull request as draft May 17, 2024 08:17
@@ -124,10 +124,9 @@ class BookRepository extends ServiceEntityRepository
}
```

The generated code is not compatible with Sylius Resource yet, so we need to make few changes.
The generated code could be not compatible with Sylius Resource in some cases, so we need to make few changes.
Copy link
Member Author

@loic425 loic425 May 17, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hum I think I should move all this part on Index operation without any grid.

@loic425 loic425 force-pushed the repository branch 3 times, most recently from f118df8 to 713d537 Compare May 17, 2024 13:09
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.

1 participant