Summary

Closes #2418 (closed)

Original change on prod is that if you hit enter too quickly after posting a link, it will not post with the rich embed preview, but it will upload in a separate comment below after the comment is posted instead.

Behaviourally,

• A user should be able to post a link without a rich embed
• Once the user has posted, they should not see rich-embed in new window.

Steps to test

1. Log in and go to a post
2. Copy a link e.g. https://www.google.com/
3. Paste it into a comment on the post and rapidly hit enter (do not wait for preview response)
4. it should post without the preview, there should not be a new attachment that loads under the comment.
5. Repeat with an invalid URL e.g. https://www.ks3fnj3n9.com
6. Try typing a url
7. Ensure when you wait for the rich-embed to come back, it does post.

Estimated Regression Scope

The area that this change impacts has been reduced; it should now, in the worst-case scenario mess something up only with rich-embeds.

added scoped labels

assigned to @edgebal

added 1 commit

• 319a9bff - Removed arrow functions

Compare with previous version

added scoped label and automatically removed label

Needs @edgebal 's dev changes to be merged into master first

added 27 commits

• 319a9bff...745769a5 - 26 commits from branch master
• b70dd721 - Merge branch 'master' of gitlab.com:minds/front into fix/premature-posting-2418

Compare with previous version

approved this merge request

assigned to @benhayward.ben, @brianhatchet, and @markeharding and unassigned @edgebal

added scoped label and automatically removed label

Works OK!

Muuuuuuch cleaner <3

approved this merge request

Works.

approved this merge request

merged

mentioned in commit 4d90e094