WEB-232 Update Sitecore Back end solution to 9.3#1
WEB-232 Update Sitecore Back end solution to 9.3#1vincentlui-alinta wants to merge 5 commits intomasterfrom
Conversation
daniel-simpson-alinta
left a comment
There was a problem hiding this comment.
LGTM, though I'm amazed that there isn't something similar in the upstream repo.
Edit: I just looked at the upstream and its targeting Sitecore 8.2? 😱
|
Looks like our repo was forked from a fork... Is it worth submitting these changes as a PR back to the original repo? https://github.com/iamandycohen/UrlRewrite |
I am a bit hesitant to do a PR back into the original repo as it has not been maintained for so long. Also don't forget that ideally this module should be removed from Alinta's solution soon rather than later. I would prefer to give this PR more TLC if it's going back to the original repo. |
Fixing all the breaking changes for the Redirect module