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

Port for Rails 5.2.2 #84

Closed
wants to merge 1 commit into from
Closed

Port for Rails 5.2.2 #84

wants to merge 1 commit into from

Conversation

dhilst
Copy link

@dhilst dhilst commented Jan 26, 2019

Hi!!!

I was looking for this kind of functionality in Redmine, is so so so useful, thanks for developing this awesome plugin! I had to make some little tweaks to it run on Rails 5.2.2. I hope that you appreciate!

Cheers!

Copy link

@kou kou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I confirmed that this patch works with Redmine 3.4.8 and 4.0.1.
But I think that #80 is better than this because #80 removes needless codes.

@@ -28,7 +28,12 @@
end


if Rails::VERSION::MAJOR >= 3
if Rails::VERSION::MAJOR >= 5 and Rails::VERSION::MINOR >= 1
ActiveSupport::Reloader.to_prepare do
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can use Rails.configuration.to_prepare on both Redmine 3 and 4 like #80 did.

@dhilst
Copy link
Author

dhilst commented Feb 2, 2019

I didn't look for a PR before starting this, my fault I will close this as duplicate of #80

@dhilst dhilst closed this Feb 2, 2019
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.

2 participants