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
feat: add relation options to all tree queries #8080
Conversation
@imnotjames would be nice if you could merge this PR. As you can see many people are waiting for it. |
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.
Looks good, just one small thing about the location of the tests.
@pleerock Maybe you could merge this since many developers including me urgently need this. The longer it takes, the more likely merge conflicts will appear. |
@TheProgrammer21 thanks for waiting, it took a bit to me to reach your PR. Thank you for contribution! |
Thank you for finally merging and for your work in general! |
* feat: add relation options to all tree queries Closes: typeorm#8076 * fix: corrected array index in test case * try to fix failing test * fix: sort the array in order to be able to apply tests
Closes: #8076
Description of change
Added the
relations
option to all treeRepository methods.Pull-Request Checklist
master
branchnpm run lint
passes with this changenpm run test
passes with this changeFixes #0000
The changes are more or less the same I made in the previous pull request, just that I did it on all other methods too. I created test cases for each method, with and without the option set.