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.

Sign up for GitHub

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

Jump to bottom

EZP-31371: Copying a web page link w/o protocol in a Word paragraph leads to a 404 error #114

Merged

lserwatka merged 2 commits into master from EZP-31371-links_missing_protocol

Feb 19, 2020

Member

konradoboza commented Feb 19, 2020

Question	Answer
JIRA issue	EZP-31371
Bug/Improvement	yes
New feature	no
Target version	master
BC breaks	no
Tests pass	yes
Doc needed	no

This is targetting for eZ Platform 3.0, followup for ezsystems/ezplatform-admin-ui#1245.

TODO:

Implement feature / fix a bug.
Implement tests.
Fix new code according to Coding Standards ($ composer fix-cs).
Ask for Code Review.


          EZP-31371: Copying a web page link w/o protocol in a Word paragraph l…

085d741

…eads to a 404 error

konradoboza added Bug Ready for review labels

konradoboza requested review from GrabowskiM, dew326 and lucasOsti

February 19, 2020 10:15

konradoboza self-assigned this


          changed event to CKEditor's aferPaste

dew326 approved these changes

View reviewed changes

GrabowskiM approved these changes

View reviewed changes

konradoboza added Ready for QA and removed Ready for review labels

micszo self-assigned this

konradoboza removed their assignment

micszo approved these changes

View reviewed changes

Member

micszo left a comment

Retested successfully on eZ Platform EE master with branch.

micszo added QA approved and removed Ready for QA labels

micszo removed their assignment

lucasOsti approved these changes

View reviewed changes

konradoboza added the Ready for MERGE label

lserwatka merged commit 98bdf83 into master

lserwatka deleted the EZP-31371-links_missing_protocol branch

February 19, 2020 14:55

konradoboza assigned konradoboza and unassigned konradoboza

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug QA approved Ready for MERGE