closes minds#553

closes #2312

mentioned in merge request !374 (closed)

added scoped label

Confirmed this works. However, my copied gif no longer animates with copy/paste flow?

Also confirm !394 (diffs, comment 187720495) and then we are good

added 28 commits

• 18d4f68c...82595d15 - 26 commits from branch master
• f21330be - Merge remote-tracking branch 'upstream/master' into feat/copy-and-paste-images-in-posts
• c71558d1 - (fix): brought back e.preventDefault();

Compare with previous version

@markeharding, I did some research and found out that copying and pasting gifs only captures the current frame. It works like that everywhere, I think

resolved all discussions

added scoped label

mentioned in issue #2312

changed the description

added 539 commits

• c71558d1...80e43c8c - 538 commits from branch master
• 52206363 - Merge remote-tracking branch 'upstream/master' into feat/copy-and-paste-images-in-posts

Compare with previous version

added 1 commit

• 7d7b9624 - (fix): lint

Compare with previous version

approved this merge request

assigned to @edgebal and @xander-miller

Copy and paste into a post. Bam, it works great

Working for me too.

approved this merge request

approved this merge request

unapproved this merge request

approved this merge request

Works for me too. I couldn't delete the attachment, but it's an upstream issue.

Several failures on the e2e tests. I think these are getting fixed on other in coming merge requests though.

I currently don't have permissions to merge this into master. Assigning it to @markeharding

added scoped label and automatically removed label

assigned to @eiennohi

unassigned @edgebal and @xander-miller

added 1 deleted label

added 1 commit

• c0eeae28 - (fix): remove aliases and use correct naming for outputs

Compare with previous version

resolved all threads

added scoped label and automatically removed label

added scoped label and automatically removed label

Comments work for me but the newsfeed does not.

added 1 commit

• e0dbd1c2 - (fix): use filePaste vs OnFilePaste

Compare with previous version

@markeharding just solved. Sorry about this, I had a local issue and missed the error. Just tested and it works fine now.

added scoped label and automatically removed label

Error when trying to remove the image. This doesn't happen on production.

vendor.779a60b3ce201322a755.js:1 TypeError: Cannot read property 'unsubscribe' of undefined
    at l.abort (main.4e9e29ab7caee12f5899.js:1)
    at l.removeAttachment (main.4e9e29ab7caee12f5899.js:1)
    at Object.handleEvent (main.4e9e29ab7caee12f5899.js:1)
    at Object.handleEvent (vendor.779a60b3ce201322a755.js:1)
    at Object.handleEvent (vendor.779a60b3ce201322a755.js:1)
    at Fl (vendor.779a60b3ce201322a755.js:1)
    at vendor.779a60b3ce201322a755.js:1
    at HTMLDivElement.<anonymous> (vendor.779a60b3ce201322a755.js:1)
    at HTMLDivElement.r (vendor.779a60b3ce201322a755.js:1)
    at e.invokeTask (polyfills.055b24b90bf66d29b201.js:1)
    at Object.onInvokeTask (vendor.779a60b3ce201322a755.js:1)
    at e.invokeTask (polyfills.055b24b90bf66d29b201.js:1)
    at t.runTask (polyfills.055b24b90bf66d29b201.js:1)
    at t.invokeTask [as invoke] (polyfills.055b24b90bf66d29b201.js:1)
    at _ (polyfills.055b24b90bf66d29b201.js:1)
    at HTMLDivElement.m (polyfills.055b24b90bf66d29b201.js:1)

Confirmed that I can delete a C/Ped image without error

added scoped label and automatically removed label

approved this merge request

unapproved this merge request

added 1 deleted label and removed 1 deleted label

added 39 commits

• e0dbd1c2...f3af959d - 37 commits from branch master
• a4842dd6 - (fix): only cancel uploadSubscription if it has a value
• c4f6f110 - Merge remote-tracking branch 'upstream/master' into feat/copy-and-paste-images-in-posts

Compare with previous version

added scoped label and automatically removed label

added 28 commits

• c4f6f110...b00415cc - 27 commits from branch master
• 64521606 - Merge remote-tracking branch 'upstream/master' into feat/copy-and-paste-images-in-posts

Compare with previous version

approved this merge request

added 7 commits

• 64521606...31db9b75 - 6 commits from branch master
• 62177ec5 - Merge remote-tracking branch 'upstream/master' into feat/copy-and-paste-images-in-posts

Compare with previous version

approved this merge request

approved this merge request

approved this merge request