-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[Documentation][Bundles] SyliusGridBundle docs update #6312
Conversation
CoderMaggie
commented
Oct 5, 2016
Q | A |
---|---|
Doc fix? | yes |
New docs? | no |
BC breaks? | no |
Related tickets | partially #5275 |
License | MIT |
@TheMadeleine What about custom actions on the grid ? You should have a little sentence for actions in "main" section and the ones in "item" section :) Great thanks for your job. |
43789e2
to
356c6b4
Compare
|
||
The filter allows the user to select following search options: | ||
|
||
* contains | ||
* not contains | ||
* equal | ||
* starts with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not equal
is missing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is no not equal
:)
fields: | ||
email: | ||
type: string | ||
label: app.ui.email |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be app.ui.date
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
but... why?
@ylastapis to be honest I'm not sure about the custom actions on grids. Let me call @pjedrzejewski or @Zales0123 here 😆 |
driver: doctrine/orm # Use appropriate driver. | ||
resource: app.tournament | ||
app_admin_supplier: | ||
... | ||
sorting: | ||
name: asc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't it expected that name is an array and the path is required?
resource: app.user # Resource name. | ||
app_user: # Your grid name | ||
driver: doctrine/orm # Data source driver | ||
resource: app.user # Resource name |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be:
driver:
name: doctrine/orm
options:
class: AppBundle\Entity\User
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed now! ;)
@ylastapis We will document that as well, but actions need to be tweaked a bit, I will take care of that soon. :) |
05ab655
to
3aa14e1
Compare
Thank you Magda, nice work! |