Skip to content

Conversation

@madmaxdios
Copy link
Contributor

the default values ​​of the options are updated, and the possible values ​​of orphanedRowAction are placed in the diagram for better reading.

the default values ​​of the options are updated, and the possible values ​​of orphanedRowAction are placed in the diagram for better reading.
@jorenvandeweyer
Copy link
Contributor

@madmaxdios can you apply the feedback please?

@madmaxdios
Copy link
Contributor Author

@jorenvandeweyer srry for delay, is there anything else I need to do now?

Copy link
Collaborator

@gioboa gioboa left a 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

@madmaxdios
Copy link
Contributor Author

I think now is ok

Co-authored-by: Giorgio Boa <35845425+gioboa@users.noreply.github.com>
Copy link
Collaborator

@gioboa gioboa left a 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 ✅

@madmaxdios
Copy link
Contributor Author

madmaxdios commented Jan 24, 2025

Thanks you @gioboa for all your work.
Do I need do anything else?

@gioboa gioboa added the size-xs Trivial tasks or quick fixes. Estimated effort: Less than 1 hour label Jan 24, 2025
Copy link
Collaborator

@gioboa gioboa left a 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 👏

@michaelbromley michaelbromley merged commit 1cd3ff7 into typeorm:master Feb 14, 2025
60 of 61 checks passed
aberonni pushed a commit to global-121/typeorm that referenced this pull request Mar 5, 2025
ThbltLmr pushed a commit to ThbltLmr/typeorm that referenced this pull request Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation size-xs Trivial tasks or quick fixes. Estimated effort: Less than 1 hour

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants