peclik / clipboard_image_paste

Redmine plugin for pasting cropped image from clipboard as an attachment.
264 stars 118 forks source link

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

Closed Utopism closed 3 years ago

Utopism commented 6 years ago

Hi,

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

Thanks a lot.

kzalewski commented 5 years ago

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 commented 5 years ago

Thank kzalewski to have tested it.

peclick, can you please merge this pull request ?

canyildiz commented 5 years ago

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
alfal commented 5 years ago

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

bviktor commented 5 years ago

Merge please :)

AdamN1 commented 5 years ago

Hi, can we merge this?

petrlosa commented 5 years ago

please merge this, this work on 4.0.3 OK

jerome-salanhac commented 5 years ago

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

Utopism commented 5 years ago

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

You're welcome.

shanedobson-ibboost commented 5 years ago

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 commented 5 years ago

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

scottfu001 commented 5 years ago

thanks a lot, worked on 4.0.4 stable.

shanedobson-ibboost commented 5 years ago

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

Nico207 commented 5 years ago

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 commented 5 years ago

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 commented 4 years ago

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

bviktor commented 4 years ago

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

dsardelic commented 4 years ago

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 commented 4 years ago

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

This plugins is much better:

So please, keep it going.

hiroshi24da commented 4 years ago

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 commented 4 years ago

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

olivierb2 commented 4 years ago

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