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.

Sign up for GitHub

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

Jump to bottom

MONGOID-5608 - Allow exists? on relations #91

Draft

johnnyshields wants to merge 5 commits into master from fix-relations-exists

Contributor

johnnyshields commented Apr 20, 2023 •

edited

Loading

Port of mongodb/mongoid#5610

TODO:

Rspecs
Investigate how any?(&:persisted) should behave on embedded docs. Potentially there is a second bug with embedded criteria exists? returning non-persisted docs.


          Fix exists on relations

ba0ae5f

johnnyshields changed the title ~~MONGOID-5608 - Fix exists on relations~~ MONGOID-5608 - Allow exists? on relations

johnnyshields marked this pull request as draft

April 20, 2023 17:06

johnnyshields added 4 commits

April 21, 2023 16:25


          Fix MONGOID-5610

09aadde


          Fix rubocop

b1579e8


          Fix exists?

95a92eb


          blank? should use any?

069dc61

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet