-
Notifications
You must be signed in to change notification settings - Fork 96
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
Redmine 4.2 and Download attachments at once #86
Comments
#87 I have uploaded pull request which should fix the issue. I used following method to find difference was:
And that shows what missing:
Further more repo owner @tofi86 should consider to find some better method to provide modifications than rewriting whole template because this will always lead to similar problem when upstream changes anything. I don't know if this is possible in Ruby/Rails/Redmine but should be considered if it is. |
Also important note that this fix will make this plugin Redmine 4.2+ only, since there is any version checking to decide where button should or should not be presented. Redmine 4.2 also added controller that is required for download all functionality to zip attachments on the fly through zip stream. |
When using redmine 4.2, the button to download all attachments at once is not being shown.
The text was updated successfully, but these errors were encountered: