-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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.
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
Ibn labs/additional module directories dev 4 #5973
Ibn labs/additional module directories dev 4 #5973
Conversation
(cherry picked from commit 98e3592)
…abs/additional-module-directories
…alculate VirtualPath
rename AppLocations to ExtensionLocations remove ../ local path
Conflicts: src/Orchard/Orchard.Framework.csproj
Conflicts: src/Orchard/Orchard.Framework.csproj
Conflicts: src/Orchard.Web/Modules/Orchard.MultiTenancy/Views/Admin/Edit.cshtml
[Fixes OrchardCMS#5912] Improve warning css in multi-tenant edit
…dditional-module-directories-dev-4 Conflicts: src/Orchard.Web/Web.config
@@ -22,7 +21,9 @@ public class Compare : ValidationRule { | |||
} | |||
|
|||
private LocalizedString GetValidationMessage(ValidationContext context) { | |||
return new LocalizedString(Tokenize(T(ErrorMessage.WithDefault(String.Format("{0} must match the value of {1}.", context.FieldName, TargetName))).ToString(), context)); | |||
return String.IsNullOrWhiteSpace(ErrorMessage) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This has nothing to do with the PR
Could you try to squash your changes in a single commit, and "force push" to the same branch you are currently working on? This PR will be updated automatically. If you don't know how to do, just ping me on skype or gitter. |
#5982 is a squashed variant relative to 1.9.x . |
Almost ! |
A variant of PR 5604 - against dev