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

Updates to make plugin work with next Redmine V4.0.0 #80

Merged
merged 6 commits into from
Dec 17, 2020

Conversation

Utopism
Copy link
Contributor

@Utopism Utopism commented Nov 20, 2018

Hi,

Can you please review this pull request ?
This version is compatible with previous versions.

Thanks a lot.

…sh base class)

modifié :         lib/clipboard_image_paste/attachment_patch.rb
@kzalewski
Copy link

I can confirm that this patch works nicely with Redmine 4.0.0. Without it, a migration to Redmine v4.0 produces a bunch of errors when executing rake db:migrate.

@Utopism
Copy link
Contributor Author

Utopism commented Dec 11, 2018

Thank kzalewski to have tested it.

peclick, can you please merge this pull request ?

@canyildiz
Copy link

This worked great with the following environment:

Environment:
  Redmine version                4.0.1.stable
  Ruby version                   2.5.3-p105 (2018-10-18) [x86_64-linux]
  Rails version                  5.2.2
  Environment                    production
  Database adapter               PostgreSQL
  Mailer queue                   ActiveJob::QueueAdapters::AsyncAdapter
  Mailer delivery                smtp
SCM:
  Subversion                     1.9.5
  Mercurial                      4.0
  Bazaar                         2.8.0
  Git                            2.11.0
  Filesystem                     
Redmine plugins:
  clipboard_image_paste          1.13

@kou kou mentioned this pull request Feb 1, 2019
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.
It may be better that we add 4.0.x to supported Redmine version list in README.

lib/clipboard_image_paste/attachment_patch.rb Show resolved Hide resolved
@alfal
Copy link

alfal commented Feb 1, 2019

Great ,this plugin can run again, thanks @Utopism Utopism && @kou kou

@bviktor
Copy link

bviktor commented Feb 11, 2019

Merge please :)

README.textile Outdated Show resolved Hide resolved
modifié :         README.textile
@AdamN1
Copy link

AdamN1 commented Mar 2, 2019

Hi,
can we merge this?

@petrlosa
Copy link

please merge this, this work on 4.0.3 OK

@jerome-salanhac
Copy link

jerome-salanhac commented May 14, 2019

work fine, thank a lot ! 👍 (redmine 4.0.3)

@Utopism
Copy link
Contributor Author

Utopism commented May 14, 2019

work fine, thank a lot ! +1 (redmine 4.0.3)

You're welcome.

@shanedobson-ibboost
Copy link

This is failing for me in Redmine 4.0.4. The install and migrate works perfectly well, but when I actually attempt to click on the button to use the plugin, nothing happens. Nothing in the log either. As well as that the button isn't rendering correctly - it appears below well outside the area where it normally does.

redmine

@shanedobson-ibboost
Copy link

Actually to add to that - the 'Add picture from clipboard' button appears at the bottom of all pages on Redmine after installing this

@scottfu001
Copy link

thanks a lot, worked on 4.0.4 stable.

@shanedobson-ibboost
Copy link

shanedobson-ibboost commented Aug 27, 2019

Also working for me on 4.0.4 stable - permissions error from me was stopping it. Thanks!

@Nico207
Copy link

Nico207 commented Nov 28, 2019

This patch work in this environment:

Redmine version : 4.0.5.stable.19319
Ruby version : 2.5.1-p57 (2018-03-29) [x86_64-linux-gnu]
Rails version : 5.2.3

Thanks a lot !

@Utopism
Copy link
Contributor Author

Utopism commented Nov 28, 2019

This is failing for me in Redmine 4.0.4. The install and migrate works perfectly well, but when I actually attempt to click on the button to use the plugin, nothing happens. Nothing in the log either. As well as that the button isn't rendering correctly - it appears below well outside the area where it normally does.

redmine

Hi, did tried to remove old assets with :
RAILS_ENV=production rails assets:clean
Do you have any other plugin that could interfere with this one ?

@dsardelic
Copy link

Seems to be working OK in the following environment:
Screenshot from 2020-05-16 22-44-55.

@bviktor
Copy link

bviktor commented May 16, 2020

Redmine 4.1 has this feature built-in, so having this plugin is pretty pointless ever since.

@dsardelic
Copy link

Redmine 4.1 has this feature built-in, so having this plugin is pretty pointless ever since.

Right you are, dear sir. Feature #3816, released in v4.1.0. Totally missed it.

@borisbrodski
Copy link

Redmine 4.1 has this feature built-in, so having this plugin is pretty pointless ever since.

This plugins is much better:

  • You can crop the images
  • You can give a meaningfull names to the images

So please, keep it going.

@hiroshi24da
Copy link

hiroshi24da commented Jun 26, 2020

I agree with borisbrodski.
In addition, feature #3816 works when text formatting is markdown,
but does't work with CKEditor.
https://www.redmine.org/issues/3816#note-152

@eric-it6
Copy link

Thanks man, I think this is the only version that still works under 4.0.x. Tested under 4.0.6.

@olivierb2
Copy link

+1, working for me on redmine 4.0.7 (Debian Buster Backports official packages)

@peclik peclik merged commit 9c8330d into peclik:master Dec 17, 2020
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.