Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: support Relation->orderBy() #191

Closed
wants to merge 1 commit into from
Closed

fix: support Relation->orderBy() #191

wants to merge 1 commit into from

Conversation

caugner
Copy link
Contributor

@caugner caugner commented Jul 17, 2021

Fixes #190.

⚠️ Depends on vimeo/psalm#6060 (merged, released in vimeo/psalm 4.9.0)

@caugner caugner marked this pull request as draft July 17, 2021 01:37
@caugner
Copy link
Contributor Author

caugner commented Mar 6, 2022

Closing this PR, because I have left the company that was using this package and for which I was doing these contributions as part of my daily work. 👋

Apologies to anybody who may be disappointed, because they counted on this PR getting merged someday. 🙏

Obviously, if anybody wants to base their contribution on this PR, feel free to do so! No attribution required.

@caugner caugner closed this Mar 6, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support hasMany(...)->orderBy()
1 participant