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
Force download of attachments #1490
Conversation
46fc0cc
to
5ab8eb5
Compare
Gemfile
Outdated
@@ -119,7 +119,6 @@ group :development do | |||
gem 'binding_of_caller' # enhances better_errors | |||
|
|||
# static code analyzer | |||
gem 'rubocop', require: false |
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.
you might have accidentally removed rubocop from the gemfile
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.
Could you fix Gemfile and Gemfile.lock? Otherwise, LGTM!
b54838d
to
20cc546
Compare
Restored the old Gemfiles, previously I was able to have rubocop working with just rubocop-rails and so thought it superseded rubocop |
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.
LGTM!
Description
Cause attachments to be downloaded instead of being opened with the default MIME handler.
I had to also fix the rubocop-rails dependency in the Gemfile.lock as the previous config broke rubocop for me.
Motivation and Context
How Has This Been Tested?
Types of changes
Checklist:
overcommit --install && overcommit --sign
to use pre-commit hook for lintingOther issues / help required
If unsure, feel free to submit first and we'll help you along.