removed redundant branch in UserAction, updated Changelog

Author: isc-rsaptars

CHANGELOG.md

@@ -11,6 +11,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Web UI includes a "Push Branch" button for local branches that are ahead of upstream
 
 ### Fixed
+- Fixed empty mappings when SourceControl.Git.Settings is instantiated (#250)
 - Studio export path doesn't get weird mixed slahes on Windows (#252)
 - Fixed bug with adding mappings through the Settings page (#270)
 - Pulling add/delete of multiple non-IRIS files no longer causes error (#273)

cls/SourceControl/Git/Utils.cls

@@ -233,11 +233,6 @@ ClassMethod UserAction(InternalName As %String, MenuName As %String, ByRef Targe
         set ec = ..AddToSourceControl(InternalName)
     } elseif (menuItemName = "RemoveFromSC") {
         set ec = ..RemoveFromSourceControl(InternalName)
-    } // is there any reason this code block is repeated?
-    elseif (menuItemName = "Commit") {
-        set Target = "Please enter a commit message"
-        set Action = 7
-        quit $$$OK
     }
     quit ec
 }