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.

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

Auto-include Responders module if responders gem is present #30

Open
wagenet opened this issue May 28, 2019 · 1 comment
Open

Auto-include Responders module if responders gem is present #30

wagenet opened this issue May 28, 2019 · 1 comment
Labels
enhancement New feature or request

Comments

@wagenet
Copy link
Collaborator

wagenet commented May 28, 2019 •
edited
Loading

This is something that might be worth doing. We should also consider any potential downsides.
@wagenet wagenet added the enhancement New feature or request label May 28, 2019
@wagenet wagenet changed the title Consider auto-including Responders module if responders gem is present Auto-include Responders module if responders gem is present May 28, 2019
@wagenet wagenet mentioned this issue May 28, 2019
Closed
@wagenet
Copy link
Collaborator Author

wagenet commented Jun 13, 2019

We wouldn't want to auto-include as-is, since it adds a call to "respond_to" that might not be correct in all circumstances.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@wagenet