Skip to content
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

Add plucks functionality for followers of an object #102

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

rlgreen91
Copy link

Adds functionality simulating the Activerecord method pluck. Like pluck, the method can accept multiple column names.

The method currently returns data for all followers of the object, regardless of the scope. Trying to allow options on scoping causes a problem with eager loading as this is a polymorphic association. I chose to abandon that route as I did not want to change a fundamental feature of the gem as it is now. If anyone knows of a way to allow options on scoping without running into eager loading efforts, please let me know of add your own commits to this branch.

@rlgreen91
Copy link
Author

Hmm, it looks like the issue is with minitest plugin, but that seems to work fine on my end.

rlgreen91 added 3 commits May 19, 2018 13:41
…orts plugin. This addition addresses that bug without changing the versions listed in the Gemfile or gemspec.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant