-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
Update relations.md #11161
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
Update relations.md #11161
Conversation
the default values of the options are updated, and the possible values of orphanedRowAction are placed in the diagram for better reading.
|
@madmaxdios can you apply the feedback please? |
|
@jorenvandeweyer srry for delay, is there anything else I need to do now? |
gioboa
left a comment
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.
it looks great to me 🎉
My only concern is about the default syntax
In this way is more understandable
`eager: boolean` (default: `false`)
We are more than happy to merge this PR if you fix this small thing.
Thanks for your help @madmaxdios
|
I think now is ok |
Co-authored-by: Giorgio Boa <35845425+gioboa@users.noreply.github.com>
gioboa
left a comment
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.
Thanks @madmaxdios well done ✅
|
Thanks you @gioboa for all your work. |
gioboa
left a comment
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.
Thanks you @gioboa for all your work.
Do I need do anything else?
The next step is the review form a member of the maintainer group.
This PR is straightforward IMHO 👏
the default values of the options are updated, and the possible values of orphanedRowAction are placed in the diagram for better reading.