Skip to content

Comments

Redmine 5 compatible#5

Open
jcatrysse wants to merge 2 commits intoThatCompany:masterfrom
redminetrustteam:master_5
Open

Redmine 5 compatible#5
jcatrysse wants to merge 2 commits intoThatCompany:masterfrom
redminetrustteam:master_5

Conversation

@jcatrysse
Copy link

Add a GEM + small code change to make it work with Redmine 5.
Resolve popup error.

@nekrasovdmitriy
Copy link

Thank you, for those who will try to use these commits, do not forget to bundle add render_parent from redmine root.

@jcormier
Copy link

Thank you, for those who will try to use these commits, do not forget to bundle add render_parent from redmine root.

Could you expand on this?

@nekrasovdmitriy
Copy link

To make these commits work with Redmine 5, you have to install additional Gem "render_parent". I described how to do this from your Redmine installation.

@jcormier
Copy link

Ahh is there a file that needs to get updated so that dependency is installed through the normal plugin install process. Without having to explicitly call out render_parent

@jcatrysse
Copy link
Author

jcatrysse commented May 31, 2024

Ahh is there a file that needs to get updated so that dependency is installed through the normal plugin install process. Without having to explicitly call out render_parent

This shouldn't normally be executed manually. As gem 'render_parent', '~> 0.1.0' has already been added to the init.rb Gemfile file.

Simply running bundle install should do the trick.

@jcormier
Copy link

This shouldn't normally be executed manually. As gem 'render_parent', '~> 0.1.0' has already been added to the init.rb file.

Perhaps that's missing, I don't see it in this commit or in the master branch.

@jcatrysse
Copy link
Author

This shouldn't normally be executed manually. As gem 'render_parent', '~> 0.1.0' has already been added to the init.rb file.

Perhaps that's missing, I don't see it in this commit or in the master branch.

It's not on the master branch but in Pull Request number #5

Here are the details (two commits): https://github.com/ThatCompany/that_attachments_limit/pull/5/commits

@jcormier
Copy link

Okay sorry I see it, it's in Gemfile and not init.rb

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.

3 participants